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

Disable smoketests #76

Merged
merged 2 commits into from
Jul 30, 2020
Merged

Conversation

jillr
Copy link
Collaborator

@jillr jillr commented May 18, 2020

SUMMARY

Smoketests are not needed in collections, and none exist in this
collection anyway.
ansible-collections/overview#45 (comment)

amazon.aws will be using the pip package, so keep this one consistent.
We won't be using codecov-python, does not seem to be a drop-in replacement after all.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

tests/utils/shippable/aws.sh
tests/utils/shippable/shippable.sh

@jillr jillr changed the title Disable smoketests and use codecov pip package Disable smoketests May 19, 2020
@jillr jillr changed the base branch from master to main July 2, 2020 19:48
jillr added 2 commits July 28, 2020 11:47
Smoketests are not needed in collections, and none exist in this
collection anyway.
ansible-collections/overview#45 (comment)
@jillr jillr force-pushed the disable_smoketests branch from 6d444a8 to 9526609 Compare July 28, 2020 18:48
@jillr
Copy link
Collaborator Author

jillr commented Jul 29, 2020

@felixfontein would you be able to give this shippable.sh change a look over?

Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jillr jillr merged commit f509f74 into ansible-collections:main Jul 30, 2020
@jillr jillr deleted the disable_smoketests branch February 24, 2021 17:26
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Disable smoketests and use codecov pip package

Smoketests are not needed in collections, and none exist in this
collection anyway.
ansible-collections/overview#45 (comment)

* Not so fast on using codecov-python, not working at all in amazon.aws.
Stick with what works.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* Disable smoketests and use codecov pip package

Smoketests are not needed in collections, and none exist in this
collection anyway.
ansible-collections/overview#45 (comment)

* Not so fast on using codecov-python, not working at all in amazon.aws.
Stick with what works.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…ions#76)

* Add link to ansible-maintained contributor guide etc

* add 2.9 deprecation note
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