-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Can Changelog link to the PR? #128
Comments
I support the idea for all our repos @rubocop/rubocop-rspec WDYT? |
I like the idea too. The main downside is that you don’t know the PR number when you write the changelog entry. It would be more efficient maybe to add the links right before releasing a new version? |
You don't? Doesn't the line get added in the same PR as the change, like in this one? #99 I haven't contributed here so I don't know, just curious based on what I thought the process was. |
Yes, the changelog lines get added in each PR. What I meant was that the contributor don’t know the number of the PR until it’s opened, which is after they push their code. So if we require the PR number to be in the changelog before we can merge a PR, each contributor has to push to their branch twice: Once before opening the PR and (at least) once after opening the PR. What I’m saying is that I would very much like to still be able to make one-push pull requests. Update: Just to be clear, I like the idea, and I often find myself navigating from the RuboCop changelog to a code search to find the PR where code was introduced. Your suggestion would make that so much easier. We just need to figure out how to do it efficiently, and not put an extra burden on contributors. |
In the changelog, can it link to the PR that made the change? It would make it a lot easier to find some of these to better see the impact it is having.
The text was updated successfully, but these errors were encountered: