Bump honnef.co/go/tools from 0.4.7 to 0.8.1 in /tools - #203
Open
dependabot[bot] wants to merge 1 commit into
Open
dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Conversation
dependabot
Bot
added
dependencies
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Contributor
|
@dependabot rebase |
Bumps [honnef.co/go/tools](https://github.com/dominikh/go-tools) from 0.4.7 to 0.8.1. - [Release notes](https://github.com/dominikh/go-tools/releases) - [Commits](dominikh/go-tools@v0.4.7...v0.8.1) --- updated-dependencies: - dependency-name: honnef.co/go/tools dependency-version: 0.6.1 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
changed the title
dependabot
Bot
force-pushed
the
dependabot/go_modules/tools/honnef.co/go/tools-0.6.1
branch
from
September 17, 2026 19:59
0d772d7 to
2e7461d
Compare
justinhwang
added a commit
that referenced
this pull request
Sep 19, 2026
The CI matrix (1.17/1.18/1.19) had aged out of upstream support, and the
pinned lint tooling no longer builds on any current toolchain:
golang.org/x/tools@v0.12.1/internal/tokeninternal.go:78:9:
invalid array length -delta * delta (constant -256 of type int64)
Move the matrix to 1.25/1.26/1.27, with lint running on 1.27. Raise the
module's minimum Go version only as far as the source actually requires:
Pointer[T] wraps sync/atomic.Pointer[T], which landed in 1.19, so 1.20 is
the floor. Neither dependency constrains this further -- testify v1.12.1
declares go 1.17 and go.yaml.in/yaml/v3 declares go 1.16.
Bump honnef.co/go/tools to v0.8.1 so staticcheck and golint compile
again. staticcheck v0.8.1 declares `go 1.26.0`, so the tools module sits
well ahead of the library; it is build-only and does not affect
consumers. This supersedes #203.
With a 1.20 floor, the pointer build variants are dead code: the
`go1.18 && !go1.19` constraint is unsatisfiable, so delete that file, and
drop the now-vacuous go1.18/go1.19 tags from the remaining files. The
surviving Pointer[T] is the sync/atomic.Pointer[T] implementation that
every supported toolchain already selected.
Note that CI does not build on 1.20-1.24, so the declared floor is a
compatibility promise rather than a tested configuration.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
justinhwang
added a commit
that referenced
this pull request
Sep 21, 2026
The CI matrix (1.17/1.18/1.19) had aged out of upstream support, and the
pinned lint tooling no longer builds on any current toolchain:
golang.org/x/tools@v0.12.1/internal/tokeninternal.go:78:9:
invalid array length -delta * delta (constant -256 of type int64)
Move the matrix to 1.25/1.26/1.27, with lint running on 1.27. Raise the
module's minimum Go version only as far as the source actually requires:
Pointer[T] wraps sync/atomic.Pointer[T], which landed in 1.19, so 1.20 is
the floor. Neither dependency constrains this further -- testify v1.12.1
declares go 1.17 and go.yaml.in/yaml/v3 declares go 1.16.
Bump honnef.co/go/tools to v0.8.1 so staticcheck and golint compile
again. staticcheck v0.8.1 declares `go 1.26.0`, so the tools module sits
well ahead of the library; it is build-only and does not affect
consumers. This supersedes #203.
With a 1.20 floor, the pointer build variants are dead code: the
`go1.18 && !go1.19` constraint is unsatisfiable, so delete that file. The
surviving Pointer[T] is the sync/atomic.Pointer[T] implementation that
every supported toolchain already selected. With no build tags left, the
go118/go119 split is meaningless, so merge the two survivors into a single
pointer.go.
Note that CI does not build on 1.20-1.24, so the declared floor is a
compatibility promise rather than a tested configuration.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
justinhwang
added a commit
that referenced
this pull request
Sep 21, 2026
The CI matrix (1.17/1.18/1.19) had aged out of upstream support, and the
pinned lint tooling no longer builds on any current toolchain:
golang.org/x/tools@v0.12.1/internal/tokeninternal.go:78:9:
invalid array length -delta * delta (constant -256 of type int64)
Move the matrix to 1.25/1.26/1.27, with lint running on 1.27. Raise the
module's minimum Go version only as far as the source actually requires:
Pointer[T] wraps sync/atomic.Pointer[T], which landed in 1.19, so 1.20 is
the floor. Neither dependency constrains this further -- testify v1.12.1
declares go 1.17 and go.yaml.in/yaml/v3 declares go 1.16.
Bump honnef.co/go/tools to v0.8.1 so staticcheck and golint compile
again. staticcheck v0.8.1 declares `go 1.26.0`, so the tools module sits
well ahead of the library; it is build-only and does not affect
consumers. This supersedes #203.
With a 1.20 floor, the pointer build variants are dead code: the
`go1.18 && !go1.19` constraint is unsatisfiable, so delete that file, and
drop the now-vacuous go1.18/go1.19 tags from the remaining files. The
surviving Pointer[T] is the sync/atomic.Pointer[T] implementation that
every supported toolchain already selected.
Note that CI does not build on 1.20-1.24, so the declared floor is a
compatibility promise rather than a tested configuration.
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Bumps honnef.co/go/tools from 0.4.7 to 0.8.1.
Commits
1285a6aVersion 2026.2.1 (v0.8.1)41fe1c7website: add 2026.2.1 release notes5bb0735SA4023: only flag possibly typed nils8d63ae3go/types/typeutil: rename IsPointerLike to MaybePointerLikef01b3d2go/types/typesutil: return true from IsPointerLike for unconstrained type setsf0c0ad6Version 2026.2 (v0.8.0)0a6410fwebsite: update shell.nix81d37ddwebsite: finalize 2026.2 release notesf4f8a57analysis/facts/nilness: handle 'recover' builtinc6603d6go/ir: fix use of GO111MODULE in tests