Update docu about how to write release notes and fix commit msg from elektrabot

[4ydan]

I investigated a bit (with the help of @atmaxinger) why the libelektra release notes are not fully populated.

Steps to Reproduce the Problem

Follow RELEASE.md

Expected Result

Fully populated release notes.

Actual Result

Unfinished release notes. Note the <<scripts/generate-news-entry>>.

Tasks

• [ ] fix broken regex (see below)
• [ ] change commit msg: "release: add hashsums and statistics to release notes" to be more true
• [ ] The written command there is also wrong: scripts/git-release-stats 0.11.0.VER-1 0.11.0 must be scripts/git-release-stats v0.10.0 v0.11.0
• [ ] But they need to manually improved anyway (there are a few duplicates and we don't need to honor Restyled.io). So IMHO it actually makes sense to do this manually. So we only need to improve the documentation about that.
• [ ] "This release did not happen yet." also needs to be manually removed.
• [ ] The reason that it is not shown in RSS feed is probably that scripts/generate-news-entry also wasn't executed yet.

[markus2330]

Thank you for looking into it. Don't use too much time on it, we can also do it manually. Statistics are not the most important thing.

[markus2330]

Or maybe someone else knows what is happening why the release notes were not updated, maybe @robaerd?

Or @tmakar can you maybe answer the question if the release note were fully populated for 0.10.0 or did you do something manually?

[4ydan]

I found this regex a bit suspicious, how does it work for 0.10 and 0.11 when they are not in 0-9? Or does it only match the first digit? https://github.com/ElektraInitiative/libelektra/blob/dbbe4ae4f669c322a8f95f59112d3f5fc370bbd9/scripts/release/release.sh#L123

My guess would also be that the <<scripts/generate-news-entry>> is not automatically generated but manually inserted.

[robaerd]

Or maybe someone else knows what is happening why the release notes were not updated, maybe @robaerd?

As far as I can tell everything worked as it was supposed to? Replacing <<scripts/generate-news-entry>> was IIRC always a manual pre-release step done by @mpranj.

I found this regex a bit suspicious, how does it work for 0.10 and 0.11 when they are not in 0-9? Or does it only match the first digit?

Although the regex is wrong and should be fixed, it still worked during the release. The last . matches the second digit of the two-digit version number 0.11 and [0-9]* of the regex is just ignored.

[mpranj]

Replacing <<scripts/generate-news-entry>> was IIRC always a manual pre-release step

Yes it was. I'm not sure if someone automated it in the mean time though.

[4ydan]

As far as I can tell everything worked as it was supposed to?

I see, so it is a manual step and we can close this. Thank you for the swift replies.

[markus2330]

I see, thx for the answers!

I agree it is a bit confusing that elektrabot pushes a commit with "release: add hashsums and statistics to release notes" when the statistics actually need to be added manually.

The written command there is also wrong: scripts/git-release-stats 0.11.0.VER-1 0.11.0 must be scripts/git-release-stats v0.10.0 v0.11.0

Given the v0.11.0 tag on dbbe4ae4f669c322a8f95f59112d3f5fc370bbd9, generating the statistics works for me:

Number commits: 491
      1 Author: Adi <[email protected]>
      1 Author: Maximilian Irlinger <[email protected]>
      1 Author: Mihael Pranjic <[email protected]>
      1 Author: Stefan Hanreich <[email protected]>
      2 Author: Klemens Böswirth <[email protected]>
      2 Author: tucek <[email protected]>
      3 Author: Aydan Namdar Ghazani <[email protected]>
      3 Author: Michel Tucek <[email protected]>
      4 Author: elektrabot <[email protected]>
      7 Author: 0x6178656c <[email protected]>
      8 Author: Hannes Laimer <[email protected]>
     10 Author: tmakar <[email protected]>
     18 Author: Markus Raab <[email protected]>
     38 Author: Restyled.io <[email protected]>
     68 Author: Florian Lindner <[email protected]>
    120 Author: Richard Stöckl <[email protected]>
    204 Author: Maximilian Irlinger <[email protected]>
 292 files changed, 20437 insertions(+), 2288 deletions(-)

But they need to manually improved anyway (there are a few duplicates and we don't need to honor Restyled.io). So IMHO it actually makes sense to do this manually. So we only need to improve the documentation about that.

"This release did not happen yet." also needs to be manually removed.

The reason that it is not shown in RSS feed is probably that scripts/generate-news-entry also wasn't executed yet.

@atmaxinger what was the error you got from scripts/git-release-stats v0.10.0 v0.11.0 ? Can you fix the remaining problems and update the docu about that?

[github-actions[bot]]

I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue. Thank you for your contributions :sparkling_heart: