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
contrib: Improve release script guard rails #16936
contrib: Improve release script guard rails #16936
Conversation
joestringer
commented
Jul 20, 2021
- Print help if zero args are provided
- Pick up on invalid URLs with a simple regex to avoid failing early
- Add the actions URL to the commit and PR messages.
* Print help if zero args are provided * Pick up on invalid URLs with a simple regex to avoid failing early * Add the actions URL to the commit and PR messages. Signed-off-by: Joe Stringer <joe@cilium.io>
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.
Although the changes are pretty straight forward, I don't have experience with using the script so my review might be limited in value.
👍 thanks for the review. I'm not too worried about the impacts, I did try it out during the last release cycle, seemed to do the right thing. All of these release scripts are just slowly getting better incrementally through these individual steps. |