-
-
Notifications
You must be signed in to change notification settings - Fork 585
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
ci: enforce commit message convention, fixes #5037 [skip ci] #5038
ci: enforce commit message convention, fixes #5037 [skip ci] #5038
Conversation
gilbertsoft
commented
Jun 28, 2023
•
edited by gitpod-io
bot
edited by gitpod-io
bot
- Use standardized PR titles, resulting in standardized master-branch commit messages #5037
Download the artifacts for this pull request:
See Testing a PR |
Please make sure to lead the docs (and error messages from github actions) with several examples of valid titles. Most of us can't read through the requirements, we just want to do it right. Examples will help a lot. |
0cb407b
to
d0c32ec
Compare
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.
Looking good. I'm concerned that we haven't run this on master yet, have you run these on master on your fork? If not, please experiment a bit on your fork with these on master.
c0f51a8
to
7af361d
Compare
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.
Sounds good, thanks!
Please make sure to test to your satisfaction on your fork before pulling.
7af361d
to
ae92fc8
Compare
Ready to be merged once @mattstein approved. Tested with Randy on my cloned repo and works like expected. Next step will be prepare release drafts based on the new labels. |