Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Title missing because of parenthesis? #9

Open
yanokwa opened this issue Jul 3, 2019 · 1 comment
Open

Title missing because of parenthesis? #9

yanokwa opened this issue Jul 3, 2019 · 1 comment

Comments

@yanokwa
Copy link
Member

yanokwa commented Jul 3, 2019

I ran this command:

cd collect;
release-notes-generator generate . v1.22.4 v1.23.0-beta.0

And for this PR getodk/collect#3159, the output was:

- (#3159)
  - Grzegorz Orczykowski @grzesiek2010 (Nafundi)

The output should be:

- Avoid ArrayIndexOutOfBoundsException in initMapPrefs() (#3159)
  - Grzegorz Orczykowski @grzesiek2010 (Nafundi)
@ggalmazor
Copy link
Contributor

On merge commits like the one in getodk/collect#3159 we use the first line of the commit message which, in this case, it's empty.

I don't think we can do much on this end to solve this particular situation. When merging PRs, we should leave the message Github creates, which normally includes the name of the PR in the first line.

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

No branches or pull requests

2 participants