-
Notifications
You must be signed in to change notification settings - Fork 17
fix(deps): update module github.com/golangci/golangci-lint/v2 to v2.8.0 #1366
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
base: main
Are you sure you want to change the base?
Conversation
ℹ️ Artifact update noticeFile name: tools/go.modIn order to perform the update(s) described in the table above, Renovate ran the
Details:
|
…ve lint errors) Updated slice initializations in multiple files to use make with a specified capacity, improving performance and clarity. Adjusted test and handler files to reflect these changes.
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
| if opt.token != "" { | ||
| headers["Authorization"] = fmt.Sprintf("Bearer %s", opt.token.Raw()) | ||
| } | ||
| opts = append(opts, otlptracehttp.WithHeaders(headers)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh you can't append here, you're making it length longer with this, you'll need to do
opts[1] = otlptracehttp.WithHeaders(headers)
(and opts[0].... above too)
Having said that, I don't know why the linter is complaining, it's more readable before!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh actually, colour me wrong! This works! Though i still think it was more readable before 😅
I got confused - as this wouldn't have worked with
opts = make([]otlptracehttp.Option, 2)
(without that extra 0!)
|
It looks like the suggestion from the linter here may be the result of a bug: alexkohler/prealloc#62 |
This PR contains the following updates:
v2.7.2→v2.8.0Release Notes
golangci/golangci-lint (github.com/golangci/golangci-lint/v2)
v2.8.0Compare Source
Released on 2026-01-07
godoc-lint: from 0.10.2 to 0.11.1 (new rule:require-stdlib-doclink)golines: from442fd00to 0.14.0gomoddirectives: from 0.7.1 to 0.8.0gosec: fromdaccba6to 2.22.11 (new rule:G116)modernize: from 0.39.0 to 0.40.0 (new analyzers:stringscut,unsafefuncs)prealloc: from 1.0.0 to 1.0.1 (message changes)unqueryvet: from 1.3.0 to 1.4.0 (new options:check-aliased-wildcard,check-string-concat,check-format-strings,check-string-builder,check-subqueries,ignored-functions,sql-builders)go-critic: from 0.14.2 to 0.14.3go-errorlint: from 1.8.0 to 1.9.0govet: from 0.39.0 to 0.40.0protogetter: from 0.3.17 to 0.3.18revive: add missing enable-default-rules settingConfiguration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.