When I created a release I had to go through all pull requests and copy the release notes from the description. Then I was wondering if there is a better way.
In another project, we use git log production/master.. --merges
for a quick overview of what we are about to deploy. Doing that with the OFN codebase gives me a rough idea as well, but it would be even better to see the release notes in there. Currently we see something like this:
commit 8a3f621b42dfa1248d6b647bcf53befe83790c28
Merge: 7cac4637f dc5302ca0
Author: Pau Pérez Fabregat <saulopefa@gmail.com>
Date: Thu Oct 25 18:20:42 2018 +0200
Merge pull request #2893 from luisramos0/deleted_products_break_inventory
Fix bug in inventory management page
commit 7cac4637fe566c32e17402ed3cf0b5ad7a19443d
Merge: acb8ec772 c9784a5ed
Author: Pau Pérez Fabregat <saulopefa@gmail.com>
Date: Thu Oct 25 18:18:53 2018 +0200
Merge pull request #2916 from luisramos0/date-transl-fix
Add Portuguese and German to list of momentJS locales
commit acb8ec772ec08c5c1b7239bd70707017dda1c432
Merge: 3cab3d179 cd5c23993
Author: Maikel <maikel@email.org.au>
Date: Thu Oct 25 14:48:51 2018 +1100
Merge pull request #2922 from mkllnk/update-rubocop-todo
Update rubocop todo list
Imagine we would copy the release notes in the merge commit message when we merge. Git could give us all the release notes with one command. It could look like this:
> git log v1.21.0.. --merges
commit 8a3f621b42dfa1248d6b647bcf53befe83790c28
Merge: 7cac4637f dc5302ca0
Author: Pau Pérez Fabregat <saulopefa@gmail.com>
Date: Thu Oct 25 18:20:42 2018 +0200
Merge pull request #2893 from luisramos0/deleted_products_break_inventory
Fix bug in inventory management page
Fixed: Fix broken inventory page. When a producer deletes a permission for a hub to add producer's products to the hub's inventory, the hub's inventory items are now being correctly updated with a revoked stamp that will not break the inventory page anymore.
commit 7cac4637fe566c32e17402ed3cf0b5ad7a19443d
Merge: acb8ec772 c9784a5ed
Author: Pau Pérez Fabregat <saulopefa@gmail.com>
Date: Thu Oct 25 18:18:53 2018 +0200
Merge pull request #2916 from luisramos0/date-transl-fix
Add Portuguese and German to list of momentJS locales
Fixed: Fixed Portuguese and German translation problem in the shopfront order cycle closure time.
commit acb8ec772ec08c5c1b7239bd70707017dda1c432
Merge: 3cab3d179 cd5c23993
Author: Maikel <maikel@email.org.au>
Date: Thu Oct 25 14:48:51 2018 +1100
Merge pull request #2922 from mkllnk/update-rubocop-todo
Update rubocop todo list
Ignore for release notes.
If this goes well, we could even automated the generation of the commit message. What do you think about this practice? We could also agree on putting the release not line in the title of the pull request. The title is added to the commit message by Github and then we don’t need to copy and paste anything.