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

Use release-drafter to draft release notes #653

Merged
merged 2 commits into from
Dec 14, 2020
Merged

Use release-drafter to draft release notes #653

merged 2 commits into from
Dec 14, 2020

Conversation

Nuru
Copy link
Contributor

@Nuru Nuru commented Dec 13, 2020

what

  • Use release-drafter to automatically generate release notes combining multiple automatic upgrades and manual changes

why

  • More consistent release notes more easily including all automated updates in a single release

notes

  • This does not automatically create a release, just draft the release notes. Releases are still done manually, and release notes and version numbers can be edited before release.

@Nuru Nuru requested a review from a team as a code owner December 13, 2020 07:54
Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

Please use GITHUB_TOKEN instead like our other release actions and rename the config for consistency.

@Nuru
Copy link
Contributor Author

Nuru commented Dec 13, 2020

Please use GITHUB_TOKEN instead like our other release actions and rename the config for consistency.

@osterman All our auto-release workflows use PUBLIC_REPO_TOKEN. See the template at build-harness/templates. This action is exactly the template action except here I set publish: false.

I will change the names if you want. My thinking was that since we use auto-release to automatically make releases, and this is not doing that, I should name it differently in order to avoid confusion. My fear is that if it is named "auto-release" people will see that and think that they do not need to manually cut a release.

@Nuru Nuru requested a review from osterman December 13, 2020 23:28
@github-actions github-actions bot added the shell label Dec 14, 2020
@osterman
Copy link
Member

osterman commented Dec 14, 2020

@Nuru that changed recently and must have slipped through code review. I would prefer we use the built-in tokens.

@osterman
Copy link
Member

The version in build-harness that I added had GITHUB_TOKEN, then it was changed in cloudposse/build-harness#249 without explanation.

@Nuru
Copy link
Contributor Author

Nuru commented Dec 14, 2020

The version in build-harness that I added had GITHUB_TOKEN, then it was changed in cloudposse/build-harness#249 without explanation.

@osterman It was copied from here, with the explanation:

osterman:
Aha so one GitHub action can not trigger another GitHub action for normal events unless using the PUBLIC_REPO_ACCESS_TOKEN

So that explains why the auto release was not triggering build on release created

And later, regarding the use if "v" in version numbers, but regarding the same PR and auto-release action

osterman:
unfortunately, this should be fixed in the upstream build-harness template which we copy everywhere

@Nuru Nuru merged commit 19beec2 into master Dec 14, 2020
@Nuru Nuru deleted the draft-release branch December 14, 2020 08:09
leb4r pushed a commit that referenced this pull request Jan 25, 2022
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