-
Notifications
You must be signed in to change notification settings - Fork 4
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
Uncoupling summaries from pull requests? #45
Comments
This is why I really like the towncrier approach to changelog, those files are easily editable and belong to the git history. |
The multiple section suggestion looks good, though the linking one could be problematic if it links to 1) pr with multiple entries, 2) some rewording of the original would be needed due to the new pr, 3) the original entry changes along the line |
Agreed on the approach taken by towncrier, and it's also the one I'd prefer personally for mature projects. In my mind changelist fills the niche between using heavy artillery like towncrier, and GitHub's generated changelog. So I think this suggestion could help with moving changelist a bit closer to towncrier without having the arguably higher barrier of adding files (which totally has advantages too!). I think we should definitely add a section to the README with alternatives and how changelist defines its scope in regards to those alternatives. That would help me with feature scoping and decision making. |
As long as it is optional, this is fine with me. My interest is in making this as simple and as automated as possible. If this gets to complicated, I would likely just move back to https://github.com/github-changelog-generator/github-changelog-generator. |
I'd address most of these failure modes with sanity checks in the implementation, which isn't ideal. And yes, most of these cases would require editing already merge previous PR descriptions, which is easy to do. |
One critique of the approach taken by changelist, is that the relation between pull requests and a summary (a single bullet point) in the release notes isn't always 1-to-1. E.g.
I've been thinking about how we could address these concerns. For 1pr-to-many, we could allow multiple
release-note
blocks in a pull request description. The only question would be, how to encode the labels for each of these items. Maybe something like (ignore escaping\
)This would basically override the pull request title (as it already does) and additionally the labels. Personally I'd like to have this option to address cases where the pull requests scoping is difficult.
Still brainstorming an approach to account for many-to-1sum. Maybe something like
I am not settled on the syntax or details yet. But I am curious what you think?
The text was updated successfully, but these errors were encountered: