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

Restore voting rights to ansible-galaxy-importer on community.aws #1587

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Jul 6, 2022

The importer job originally broke because community.aws depends on the docs fragments from amazon.aws

The docs tests have been split off into GitHub actions (which cope with the cross-repo dependency), and the docs test disabled in ansible-galaxy-importer. Since the test performs other checks, give it back the right to vote.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ ansible-tox-linters SUCCESS in 5m 11s
✔️ ansible-tox-py38 SUCCESS in 5m 34s
✔️ ansible-tox-py39 SUCCESS in 5m 03s
✔️ ansible-tox-py310 SUCCESS in 5m 00s

Copy link
Collaborator

@jillr jillr left a comment

Choose a reason for hiding this comment

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

LGTM and the job passes with a test PR

@jillr
Copy link
Collaborator

jillr commented Jul 6, 2022

@abikouo could you please review this PR and merge if you agree?

@abikouo
Copy link
Collaborator

abikouo commented Jul 7, 2022

LGTM

@abikouo abikouo added the mergeit Merge the PR (SoftwareFActory) label Jul 7, 2022
@tremble
Copy link
Contributor Author

tremble commented Jul 7, 2022

Just in case someone else wants to follow what happened:

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ ansible-tox-linters SUCCESS in 4m 49s
✔️ ansible-tox-py38 SUCCESS in 5m 06s
✔️ ansible-tox-py39 SUCCESS in 5m 02s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFActory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants