Prepare v0.17.1 release - #119
Merged
Merged
Conversation
v0.17.1 is entirely dependency work -- three `deps:` commits upgrading nine soulteary kits across new major module paths -- and nothing else has landed since v0.17.0. GoReleaser builds the release body from commit subjects, and neither prefix this repository uses for that work reaches a group: `^chore:` in the exclude list does not match `chore(deps):`, and `deps:` matches no group regexp at all. Both fall through to Others, so this release would publish as a single untitled list -- the one shape where the grouping config does nothing at exactly the release that needs it. The regexp covers both prefixes actually in this history (11 `chore(deps):` from Dependabot, 3 `deps:` by hand) plus `build(deps):`, which is Dependabot's default for gomod and would arrive the moment the commit-message config changes. Placed after Bug fixes so a `fix(deps):` still reads as a fix: GoReleaser takes the first group that matches, in definition order. Verified against Go's regexp package, which is the engine GoReleaser uses: the three release commits and both Dependabot shapes group under Dependencies, while feat/fix/perf keep the groups they had. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HarHYzcGnvPbsnzD1esjpe
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Prepares the v0.17.1 release.
What is being released
Three commits since
v0.17.0(compare), all dependency work — nine soulteary kits onto new major module paths:deps: upgrade httpcache-kit v4, logger-kit v3, metrics-kit v3— these move together because httpcache-kit's API takes logger-kit and metrics-kit types directly, and a module path is part of a type's identity. Also fixes a latent nil-interface panic:httpcache.GetDefaultMetrics()now returns theMetricsinterface and nevernil, so the old nil guard in the cache-stats API was a panic waiting to happen;UpdateCacheStats(stats)is now called unconditionally.deps: upgrade health-kit v4, middleware-kit v3, version-kit v4— Fiber support moved intofiberadaptersubpackages./healthzand/readyzkeep the localfiberHealthHandler, the same four security headers are returned, and the-Xldflags injectmain.versionrather than version-kit package variables, so the silent-devtrap in version-kit's upgrade notes does not apply here.deps: upgrade tracing-kit v2 and http-kit v2— the OTLP exporter moved to a subpackage (tracing.InitTracer→otlp.InitTracer, same arguments and return values, failure still non-fatal), and http-kit droppedClient.InjectTraceContext, which this repository never called.Nothing user-visible changes: the
apt_proxy_cache_*series, the response headers,/healthzand/readyzare all as they were. Patch rather than minor, which the repository already uses (v0.10.1,v0.5.1–v0.5.3,v0.3.1–v0.3.3).Why there is no version bump in the diff
Every version reference in this repository comes from the tag:
VERSIONin theMakefileisgit describe, GoReleaser templates the image tags from{{ .Tag }}, anddocker/DockerfiledefaultsARG VERSION=devfor local builds only. No file pins0.17.x, so there is nothing to bump — pushing the tag is sufficient.What the diff does change
GoReleaser builds the release body from commit subjects, and this release is 100% dependency commits that currently reach no group:
^chore:in the exclude list does not matchchore(deps):(Dependabot's prefix here, 11 commits in recent history).deps:(used by hand, 3 commits — all three of this release) matches no group regexp.Both fall through to
Others, so v0.17.1 would publish as one untitled list. This adds aDependenciesgroup coveringdeps:,chore(deps):andbuild(deps):, placed afterBug fixesso afix(deps):still reads as a fix — GoReleaser takes the first group that matches, in definition order.Verified against Go's
regexppackage, the engine GoReleaser uses:deps: upgrade tracing-kit v2 and http-kit v2deps: upgrade health-kit v4, middleware-kit v3, version-kit v4deps: upgrade httpcache-kit v4, logger-kit v3, metrics-kit v3chore(deps): bump …build(deps): bump …feat(proxy): …fix(proxy): …perf(cache): …refactor: …Release readiness
Run on this branch:
go build ./...— OKgo vet ./...— OKgo test -short ./...— all 12 packages passAfter merging
Note, not part of this PR
apt-proxy is the only one of these services without a
CHANGELOG.md, so the detail in these commit bodies does not survive into the release notes — GoReleaser only carries the subject lines. Adding one is a separate change and is deliberately left out of this release prep.Generated by Claude Code