-
Notifications
You must be signed in to change notification settings - Fork 146
feat(tooling,ci): add changelog formatting validation #1691
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
base: main
Are you sure you want to change the base?
feat(tooling,ci): add changelog formatting validation #1691
Conversation
2be974d
to
cd38818
Compare
Will fix the case of it flagging colons |
cd38818
to
3b3dc8e
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.
Thanks for adding this. You've added this to the tox_verify.yaml
workflow, which runs checks managed by tox.ini
. I think this is correct, but to avoid surprises in CI, it should be readily executable from tox.ini
, so it can easily be ran locally. I made a suggestion below where this could be added.
3b3dc8e
to
e76dab9
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.
Few comments below, thanks!
068a2be
to
de88d47
Compare
de88d47
to
62eeef0
Compare
- Add CI check using grep to validate bullet points end with periods - Fix existing changelog entries missing periods
62eeef0
to
c3912ae
Compare
ποΈ Description
π Related Issues
Fixes #1183
β Checklist