Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add collection-root parameter, add tests for workflow #21

Merged
merged 7 commits into from
Aug 2, 2022

Conversation

felixfontein
Copy link
Contributor

@felixfontein felixfontein commented Jul 30, 2022

The collection-root parameter is necessary for some collection repository layouts seen sometimes with collection repos for Red Hat partners. (See for example https://github.com/F5Networks/f5-ansible.) Also this is necessary to add some basic tests for this repository.

@felixfontein felixfontein changed the title Tests Add collection-root parameter, add tests for workflow Jul 30, 2022
@felixfontein
Copy link
Contributor Author

(Tests are running in felixfontein#1)

@felixfontein
Copy link
Contributor Author

The tests are now passing.

@webknjaz
Copy link
Member

RedHat partners

Please, spell "Red Hat" as two separate words, thanks.

@webknjaz webknjaz merged commit 8b55520 into ansible-community:main Aug 2, 2022
@felixfontein
Copy link
Contributor Author

@webknjaz thanks for reviewing, extending, and merging this PR. Please note that the matrix is now reduced to one element though (#21 (comment)).

@felixfontein felixfontein deleted the tests branch August 2, 2022 16:14
@webknjaz
Copy link
Member

webknjaz commented Aug 2, 2022

@webknjaz thanks for reviewing, extending, and merging this PR. Please note that the matrix is now reduced to one element though (#21 (comment)).

Yeah, I noticed that post-merge and had to push two more commits on top. This fixes it: 0d03dbf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants