-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Check downstreams to make sure that there aren't any unmerged. #1315
Conversation
I am a robot that works on MagicModules PRs! Once this PR is approved, you can feel free to merge it without taking any further steps. |
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.
minor request, either way 👍
assert len(sys.argv) == 2 | ||
unmerged = get_unmerged_prs( | ||
g, get_downstream_prs.get_github_dependencies( | ||
g, int(open(sys.argv[1]).read()))) |
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.
Can you use a local variable or comment so I know what is coming from argv[1] ?
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.
Haha, thanks.
I am a robot that works on MagicModules PRs! Once this PR is approved, you can feel free to merge it without taking any further steps. |
ed59a9d
to
888f1fe
Compare
This will make it so that we can't accidentally merge PRs with downstreams still open in the event of a transient GitHub API issue (which seems to happen sometimes). Now, merge-prs will just crash if there are open downstreams.
[all]
CI changes only.
[terraform]
[terraform-beta]
[ansible]
[inspec]