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

[v14] release: Always use latest changelog tool #47313

Merged
merged 2 commits into from
Oct 9, 2024

Conversation

camscale
Copy link
Contributor

@camscale camscale commented Oct 8, 2024

Use @latest when running the changelog tool so we do not have to keep
bumping the version (and backporting that version bump to the release
branches) when we update the tool.

Invoke the tool directly with go run instead of first installing it
with go install mostly because it's simpler, but also because the
installed tool never gets cleaned up (and due to a missing .PHONY, it
would stick to an old version).

Always run the changelog tool silently (using @ with make) so we can
easily redirect the output of the tool to a file without needing to
clean the make output. One can easily run make -n changelog to see
the command being invoked if necessary.

Backport: #47260

Use `@latest` when running the changelog tool so we do not have to keep
bumping the version (and backporting that version bump to the release
branches) when we update the tool.

Invoke the tool directly with `go run` instead of first installing it
with `go install` mostly because it's simpler, but also because the
installed tool never gets cleaned up (and due to a missing `.PHONY`, it
would stick to an old version).

Always run the changelog tool silently (using `@` with make) so we can
easily redirect the output of the tool to a file without needing to
clean the `make` output. One can easily run `make -n changelog` to see
the command being invoked if necessary.
@camscale camscale added the no-changelog Indicates that a PR does not require a changelog entry label Oct 8, 2024
@camscale camscale added this pull request to the merge queue Oct 9, 2024
Merged via the queue into branch/v14 with commit 6c27b85 Oct 9, 2024
29 of 30 checks passed
@camscale camscale deleted the bot/backport-47260-branch/v14 branch October 9, 2024 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants