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 coverage as a test requirement #457

Conversation

goneri
Copy link
Member

@goneri goneri commented Mar 4, 2021

Add the coverage module in the test-requirements.txt file. The zuul unit-tests job depends on it.

Add the `coverage` module in the test-requirements.txt file. The zuul
unit-tests job depends on it.
@goneri
Copy link
Member Author

goneri commented Mar 4, 2021

recheck

@goneri goneri merged commit ce7ccf8 into ansible-collections:main Mar 4, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
Add the `coverage` module in the test-requirements.txt file. The zuul
unit-tests job depends on it.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
Add the `coverage` module in the test-requirements.txt file. The zuul
unit-tests job depends on it.
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
Add the `coverage` module in the test-requirements.txt file. The zuul
unit-tests job depends on it.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
do not include {} boto filter by default

Depends-On: ansible-collections#480

When no filter: specified, do not include an empty dict in the list of
boto filters because it effectively matches any host.

Closes: ansible-collections#457
Closes: ansible-collections#452
ISSUE TYPE

Bugfix Pull Request

Reviewed-by: Mark Chappell <None>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants