Skip to content
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

Devel: Require updates to 'News' as part of PRs #1108

Closed
davisjam opened this issue Mar 2, 2018 · 10 comments
Closed

Devel: Require updates to 'News' as part of PRs #1108

davisjam opened this issue Mar 2, 2018 · 10 comments

Comments

@davisjam
Copy link
Contributor

davisjam commented Mar 2, 2018

Proposal type: project operations

What pain point are you perceiving?

Hard to write changelogs for new releases.

What solution are you suggesting?

Require PRs to update a "News" file with a changelog-friendly description of the change.
This is what cpython does.
Then generating a changelog is a copy/paste job or could be automated.

@davisjam
Copy link
Contributor Author

davisjam commented Mar 2, 2018

@UziTech

@UziTech
Copy link
Member

UziTech commented Mar 2, 2018

The only problem I see with making every PR add their own change to a file is whenever a PR is merged then all other unmerged PRs have a merge conflict.

I don't think the average person/dependent cares what changed and the people who do can easily compare the commits between releases

@joshbruce
Copy link
Member

To be fair, I'm more interested in the PRs that have gone into master between releases. I might be able to do the language thing pretty quickly as long as I can see the PRs easily somewhere; so, now I need the education on how to get wherever you got that page from.

I'm not sure I agree that they don't care. We've had a couple of folks talk about semver not letting them know and, from a marketing perspective, it's more about bringing on new users.

@styfle
Copy link
Member

styfle commented Mar 2, 2018

Changes since our last release: v0.3.17...master

You can run the command to fetch tags and compare only merges (PRs) on your machine like so:

git checkout master
git pull upstream master
git fetch upstream --tags
git log --merges --oneline v0.3.17..master

@joshbruce
Copy link
Member

@styfle: Thanks. I think my ignorance is greater than that. lol 😂 How do I actually get to the point of seeing that page in GitHub? UI sequence?

@UziTech
Copy link
Member

UziTech commented Mar 2, 2018

image

@davisjam
Copy link
Contributor Author

davisjam commented Mar 2, 2018

whenever a PR is merged then all other unmerged PRs have a merge conflict.

Sorry, put them in distinct news files. e.g. 2018-03-02-davisjam-PRX-ChangeLog.md.

@UziTech
Copy link
Member

UziTech commented Mar 2, 2018

@styfle
Copy link
Member

styfle commented Aug 20, 2018

I don't really see the value in this since we are already maintaining the release notes which can be fetched via GitHub API and any git user can generate a changelog from the tags.

Should we close this ticket?

@joshbruce
Copy link
Member

Tend to agree. I remember some of the conversations that led to this suggestion, namely me trying to make the release notes. This might be something worth revisiting if we ever get to the point of Python and other similar projects doing the same. Until then, the issues template and updating release notes as we merge seems to be working well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants