-
Notifications
You must be signed in to change notification settings - Fork 34
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
Generate CHANGELOG from PRs labeled with changelog #510
Conversation
Also adds a PR template with guidance on how to word PR titles.
Backend code coverage report for PR #510 |
Frontend code coverage report for PR #510 |
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.
Nice work here 😊 I'd suggest we exclude PRs authored by dependabot
from the CHANGELOG to avoid it getting messy, wdyt?
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! Can you add the updated instructions to the CONTRIBUTING file?
as far as I understood, those PR won't have the |
That's correct my bad, I misunderstood 🙃 |
@andresmgot @aangelisc yep, this excludes everything unless it has the label. |
For sure, forgot we had one 😂 |
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!
The goal of this config is to minimize discussion/decisions on what to include in the CHANGELOG when cutting a new release by moving the decision point to PR creation via the
changelog
label instead of release creation.New Flow
changelog
.