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

feat: support release notes for GitHub publishing #1044

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

njlynch
Copy link
Contributor

@njlynch njlynch commented Oct 13, 2021

Currently, the GitHub publishing job looks for a CHANGELOG file (either in a
default location or set by environment variable), parses the changelog for the
current version, and uses that result as the release notes. This change enables
an alternative approach; creating a RELEASE_NOTES file which contains the
complete release notes, ready for publishing.

The advantage of this new approach might be if you want to do some
post-processing formatting on the changelog (or even include extra notes not
normally present in the default changelog). For CDKv2, this will be used to
combine the changelog entries for the stable release (aws-cdk-lib) and for the
alpha modules.

related aws/aws-cdk#16802


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Currently, the GitHub publishing job looks for a CHANGELOG file (either in a
default location or set by environment variable), parses the changelog for the
current version, and uses that result as the release notes. This change enables
an alternative approach; creating a RELEASE_NOTES file which contains the
complete release notes, ready for publishing.

The advantage of this new approach might be if you want to do some
post-processing formatting on the changelog (or even include extra notes not
normally present in the default changelog). For CDKv2, this will be used to
combine the changelog entries for the stable release (aws-cdk-lib) and for the
alpha modules.
@rix0rrr rix0rrr merged commit 5de32d8 into main Oct 13, 2021
@rix0rrr rix0rrr deleted the njlynch/github-release-notes branch October 13, 2021 12:03
mergify bot pushed a commit to aws/aws-cdk that referenced this pull request Oct 13, 2021
…6942)

Currently, our bump process creates `CHANGELOG.md` for the `master` branch, and
`CHANGELOG.v2.md` and `CHANGELOG.v2.alpha.md` on the `v2-main` branch. For the
latter, we need to combine the two changelogs into one for the GitHub release,
where all of the artifacts of the release (both aws-cdk-lib and all the alpha
modules) are bundled together.

As proposed in the [RFC], this combined format shows the stable/aws-cdk-lib
changes first, and then the alpha module changes under their own heading.  This
generated -- but not saved -- RELEASE_NOTES.md will be used by our publishing
pipeline to generate the GitHub release notes.

[RFC]: https://github.com/aws/aws-cdk-rfcs/blob/master/text/0249-v2-experiments.md#changelog--release-notes

See companion PR - cdklabs/aws-delivlib#1044

fixes #16802

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
njlynch added a commit to aws/aws-cdk that referenced this pull request Oct 22, 2021
We now create and generate a `RELEASE_NOTES.md` file as part of our build, with
the intent to use it for our GitHub release publishing. See (#16942) and
cdklabs/aws-delivlib#1044.

What I missed is that after the build, we pack, and then only include what ends
up in `./dist` in the final build artifact that goes to publishing. This fix
includes the release notes in the dist folder so it will be picked up and used
by the release process.
mergify bot pushed a commit to aws/aws-cdk that referenced this pull request Oct 22, 2021
We now create and generate a `RELEASE_NOTES.md` file as part of our build, with
the intent to use it for our GitHub release publishing. See (#16942) and
cdklabs/aws-delivlib#1044.

What I missed is that after the build, we pack, and then only include what ends
up in `./dist` in the final build artifact that goes to publishing. This fix
includes the release notes in the dist folder so it will be picked up and used
by the release process.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
mergify bot pushed a commit to aws/aws-cdk that referenced this pull request Oct 22, 2021
We now create and generate a `RELEASE_NOTES.md` file as part of our build, with
the intent to use it for our GitHub release publishing. See (#16942) and
cdklabs/aws-delivlib#1044.

What I missed is that after the build, we pack, and then only include what ends
up in `./dist` in the final build artifact that goes to publishing. This fix
includes the release notes in the dist folder so it will be picked up and used
by the release process.

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

(cherry picked from commit 260df31)
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Feb 21, 2022
…s#16942)

Currently, our bump process creates `CHANGELOG.md` for the `master` branch, and
`CHANGELOG.v2.md` and `CHANGELOG.v2.alpha.md` on the `v2-main` branch. For the
latter, we need to combine the two changelogs into one for the GitHub release,
where all of the artifacts of the release (both aws-cdk-lib and all the alpha
modules) are bundled together.

As proposed in the [RFC], this combined format shows the stable/aws-cdk-lib
changes first, and then the alpha module changes under their own heading.  This
generated -- but not saved -- RELEASE_NOTES.md will be used by our publishing
pipeline to generate the GitHub release notes.

[RFC]: https://github.com/aws/aws-cdk-rfcs/blob/master/text/0249-v2-experiments.md#changelog--release-notes

See companion PR - cdklabs/aws-delivlib#1044

fixes aws#16802

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Feb 21, 2022
We now create and generate a `RELEASE_NOTES.md` file as part of our build, with
the intent to use it for our GitHub release publishing. See (aws#16942) and
cdklabs/aws-delivlib#1044.

What I missed is that after the build, we pack, and then only include what ends
up in `./dist` in the final build artifact that goes to publishing. This fix
includes the release notes in the dist folder so it will be picked up and used
by the release process.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
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

Successfully merging this pull request may close these issues.

2 participants