-
Notifications
You must be signed in to change notification settings - Fork 337
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
Move some tests to GH Actions #1747
Conversation
Build succeeded. ✔️ ansible-tox-linters SUCCESS in 8m 53s |
recheck |
Build succeeded. ✔️ ansible-tox-linters SUCCESS in 8m 24s |
@jillr Could you please review this PR? Ideally in combination with ansible/ansible-zuul-jobs#1803 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mariolenz Do you want me to update the branch protections after this merges for the new checks? (IMO you should have access to do this yourself for this repo. I'm happy to give you maintainer permissions on the repo but you need admin for branch protections, I'd like to double check with the community team before changing admin rights on a collection just to be in communication with them about it. What do you think?)
This looks like another discussion to to me... maybe even two discussions. But there are so many other things to do at the moment that I suggest to move it / them to the future. I don't want to loose track, you know? There are several PRs and issues I have to work on, then there is the question about the future of the CI... oh, and I have a main job, also ;-) I'll merge this PR and will backport it to the stable-2.x branch (stable-1.x is EOL so we can ignore it). Let's discuss branch protection and maintainer / admin permissions at another time. |
Build succeeded (gate pipeline). ✔️ ansible-tox-linters SUCCESS in 8m 26s |
Pull request merge failed: Resource not accessible by integration, You may need to manually rebase your PR and retry. |
community.vmware: Remove sanity and unity tests Depends-On: ansible-collections/community.vmware#1747 Remove sanity and unity tests from community.vmware. ansible-collections/community.vmware#1746 Reviewed-by: Jill R
SUMMARY
Move some tests to GH Actions.
ISSUE TYPE
COMPONENT NAME
ADDITIONAL INFORMATION
#1746