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

release: Always use latest changelog tool #47260

Merged
merged 2 commits into from
Oct 7, 2024

Conversation

camscale
Copy link
Contributor

@camscale camscale commented Oct 7, 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.

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 7, 2024
@github-actions github-actions bot requested review from kiosion and ryanclark October 7, 2024 07:03
Makefile Outdated Show resolved Hide resolved
@camscale camscale added this pull request to the merge queue Oct 7, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 7, 2024
@camscale camscale added this pull request to the merge queue Oct 7, 2024
Merged via the queue into master with commit ca91ed8 Oct 7, 2024
39 checks passed
@camscale camscale deleted the camh/use-latest-changelog-tool branch October 7, 2024 23:58
@public-teleport-github-review-bot

@camscale See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants