-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Clippy release process: Including our changelog in the upstream release announcement #3955
Comments
We could also encourage each nontrivial PR that lands to have a changelog entry in the PR body? Making changelog generation easy. |
cc @rust-lang/release I think we can, trivially, as a start, link to the clippy changelog in each new release. I've included this in T-release's agenda for next week. |
That sounds easy enough. Be sure to ping us so we can update it first 😄 Hopefully we'll always have an up to date beta changelong |
Sounds good!
The main reason I didn't include the beta changelog initially was indeed the extra work required to maintain it, but I think we can do that. |
I mean, now that I wrote the first beta changelog, it's the same amount of work -- instead of writing the release changelog at release time, write the beta changelog 😄
Yep. Before the stable release update the changelog so that the "beta" marker is removed. After the release, add a beta section. (Can be done simultaneously if you know what the beta cutoff is, which will usually be the case) |
What do y'all think about editing a changelog line into each non-refactor PR that lands? Basically, before you approve a PR, make sure you add a That way changelog generation becomes super easy. |
Yeah that sounds good. Forgetting this may be the main problem. Is there a way we can check this automatically? Maybe implement a |
We don't want this on all PRs, but yeah, a check like this is easy to add to homu. |
(we can always |
Hmm, I'd prefer to avoid too many repo-specifing things being added to homu. Maybe this can be done in the travis build itself (get the PR number, call the API to fetch the message, and fail if it's not present)? To avoid people forgetting it exists wasting CI time, a PR template could be used that adds |
Kinda wanted the fast feedback cycle of knowing immeeiately as you r+ so that the reviewer can add one (as opposed to the PR author) Iffy on adding it to the template with a default none since people may not notice it and leave it to be none |
Create PULL_REQUEST_TEMPLATE changelog: none addresses #3955 (comment)
Create PULL_REQUEST_TEMPLATE changelog: none addresses #3955 (comment)
Create PULL_REQUEST_TEMPLATE changelog: none addresses #3955 (comment)
Create PULL_REQUEST_TEMPLATE changelog: none addresses #3955 (comment)
Create PULL_REQUEST_TEMPLATE changelog: none addresses #3955 (comment)
Sadly that's not possible, since to use the GitHub API you need an API token. For this you need secrets in Travis. But Secrets aren't accessible in PRs from forks. See #4031 |
The bors CI builds are on a direct branch though, not a forked repo. I believe we do use secrets for stuff like AWS caching already. |
If the clippy team was okay with it, we could merge or copy Clippy's CHANGELOG to be another section in the Rust The relnotes tool I wrote does something similar to what is being suggested. It first finds the start and end date of the six weeks between the last release and the release before it (aka what's in beta). It uses these dates to get every PR merged to a repo (In this case |
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment))
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment))
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment))
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment))
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment)) changelog: none
We do this already |
Check for changelog entry in PR bodies cc #4031 but now on the auto and try branches. (#3955 (comment)) changelog: none
Currently Clippy maintains a changelog, however it only contains stuff for the current stable release.
We should also maintain one for the beta branch, and work with the release team each release to add things to the release blog post and link to the changelog, IMO.
Thoughts? @phansch @oli-obk
cc @Centril from the release team side
The text was updated successfully, but these errors were encountered: