Skip to content

Conversation

@JonasKunz
Copy link
Contributor

@JonasKunz JonasKunz commented Nov 19, 2024

  • Adds a new CHANGELOG.next-release.md to the repository: This file will contain unreleased changes
  • Adds a new CHANGELOG.md to the repository: This file will contain changelogs for all released versions
  • The contents of CHANGELOG.next-release.md will be prepend with a new heading for the version to CHANGELOG.md on release AND used for the GH release notes
  • In addition, the versions of relevant dependencies are appended to the release notes automatically

Example:

image

I've tested the release note generation steps using GH actions in my fork of the repo.

@JonasKunz JonasKunz requested a review from a team November 19, 2024 14:41
Comment on lines +220 to +224
echo 'notes<<RELNOTESEOF' >> $GITHUB_OUTPUT
cat CHANGELOG.next-release.md >> $GITHUB_OUTPUT
printf '\nThis release is based on the following upstream versions:\n\n' >> $GITHUB_OUTPUT
./gradlew -q printUpstreamDependenciesMarkdown >> $GITHUB_OUTPUT
echo 'RELNOTESEOF' >> $GITHUB_OUTPUT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[minor] could it be simpler to implement this completely as a gradle task ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it would be really simpler because then the gradle task would have to do the file handling, which is more code than cat CHANGELOG.next-release.md.
The multiline support code echo 'notes<<RELNOTESEOF' >> $GITHUB_OUTPUT and echo 'RELNOTESEOF' >> $GITHUB_OUTPUT would still be required, so we wouldn't save much

@JonasKunz JonasKunz merged commit e177707 into elastic:main Nov 21, 2024
16 checks passed
@JonasKunz JonasKunz deleted the changelog-generation branch November 21, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants