-
Notifications
You must be signed in to change notification settings - Fork 5
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 a Warning Annotation to GitHub Actions Logs #41
Conversation
Check if a Pull Request exists in a repository that already has a Lineage branch on the remote. If a PR does not exist emit a warning as this is undesirable behavior. This may indicate an undeleted head branch from a previously merged Lineage PR.
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.
I suggested one minor tweak to a comment; otherwise, this looks great!
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.
Smart thinking!
Please note my one small suggestion.
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.
Please see my one comment language tweak.
Reword to better describe the code that follows. Co-authored-by: Shane Frasier <jeremy.frasier@trio.dhs.gov>
Co-authored-by: dav3r <david.redmin@trio.dhs.gov>
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.
Thank you!
π£ Description
This PR adds a warning annotation to the GitHub Actions logs if a repository already has a cisagov/action-lineage branch but there is no accompanying Pull Request.
π Motivation and context
@jsf9k Noticed a repository that did not get a new Lineage PR as expected in our latest round of krakening. Upon investigation I realized that a
lineage/skeleton
branch existed and was being updated, but there was no PR attached to it for review. I thought it would be good to add this check into Lineage itself to make it easier to notice this kind of problem in the future.π§ͺ Testing
Automated tests pass. Please see this Actions run for an example run.
β Checklist