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

Bump github.com/golangci/golangci-lint from 1.56.2 to 1.57.0 in /tools #2272

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 20, 2024

Bumps github.com/golangci/golangci-lint from 1.56.2 to 1.57.0.

Release notes

Sourced from github.com/golangci/golangci-lint's releases.

v1.57.0

golangci-lint is a free and open-source project, built by people in their free time. Also, the linter authors are an important part of golangci-lint. If you use and appreciate golangci-lint please think to support us (golangci-lint maintainers and linter authors). ❤️

The most important changes are detailed here: https://golangci-lint.run/product/changelog/#v1570

⚠️ Important ⚠️

  1. Deprecated linters are inactivated, you still need to disable them if you are using enable-all.
  2. Deprecated CLI flags (about linter settings and deadline) have been removed.

Changelog

  • 2bcc0104 Revert "fix: deprecation message related to UseDefaultSkipDirs (#4533)"
  • 571f2fcf build(deps): bump github.com/Abirdcfly/dupword from 0.0.13 to 0.0.14 (#4398)
  • 4ee10a24 build(deps): bump github.com/Antonboom/testifylint from 1.1.2 to 1.1.3 (#4447)
  • 61f2f70d build(deps): bump github.com/Antonboom/testifylint from 1.1.3 to 1.2.0 (#4449)
  • 02ea91d7 build(deps): bump github.com/alexkohler/nakedret/v2 from 2.0.2 to 2.0.4 (#4540)
  • e5d2bc95 build(deps): bump github.com/catenacyber/perfsprint from 0.6.0 to 0.7.0 (#4386)
  • 24439292 build(deps): bump github.com/catenacyber/perfsprint from 0.7.0 to 0.7.1 (#4402)
  • b07bd18c build(deps): bump github.com/daixiang0/gci from 0.12.1 to 0.12.3 (#4400)
  • 9c492afc build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.0 (#4429)
  • 17e736fa build(deps): bump github.com/denis-tingaikin/go-headerfrom 0.4.3 to 0.5.0 (#4396)
  • 584520cf build(deps): bump github.com/ghostiam/protogetter from 0.3.4 to 0.3.5 (#4539)
  • f6e23692 build(deps): bump github.com/go-critic/go-critic from 0.11.1 to 0.11.2 (#4430)
  • addd06ce build(deps): bump github.com/golangci/plugin-module-register to 0.1.0 (#4510)
  • d06296eb build(deps): bump github.com/golangci/unconvert to HEAD (#4473)
  • 76ce09b5 build(deps): bump github.com/jjti/go-spancheck from 0.5.2 to 0.5.3 (#4414)
  • df70758a build(deps): bump github.com/karamaru-alpha/copyloopvar from 1.0.4 to 1.0.8 (#4444)
  • f4e33e0b build(deps): bump github.com/kunwardeep/paralleltest from 1.0.9 to 1.0.10 (#4394)
  • f8370704 build(deps): bump github.com/nunnatsa/ginkgolinter from 0.15.2 to 0.16.0 (#4530)
  • f2d41100 build(deps): bump github.com/nunnatsa/ginkgolinter from 0.16.0 to 0.16.1 (#4531)
  • 485cb6a5 build(deps): bump github.com/ryancurrah/gomodguard from 1.3.0 to 1.3.1 (#4538)
  • d36c383f build(deps): bump github.com/shirou/gopsutil/v3 from 3.24.1 to 3.24.2 (#4446)
  • b5c339f4 build(deps): bump github.com/stretchr/testify from 1.8.4 to 1.9.0 (#4448)
  • bcd015e3 build(deps): bump github.com/tomarrell/wrapcheck/v2 from 2.8.1 to 2.8.3 (#4403)
  • ce279348 build(deps): bump go-simpler.org/musttag from 0.8.0 to 0.9.0 (#4428)
  • 2c0a8ee5 build(deps): bump go-simpler.org/sloglint from 0.4.0 to 0.5.0 (#4523)
  • 6628d211 build(deps): bump golang.org/x/oauth2 from 0.17.0 to 0.18.0 in /scripts/gen_github_action_config (#4464)
  • 603e568f build(deps): bump golang.org/x/tools from 0.18.0 to 0.19.0 (#4462)
  • c6ac5a12 build(deps): bump google.golang.org/protobuf from 1.31.0 to 1.33.0 (#4503)
  • 4068bb73 build(deps): bump honnef.co/go/tools from 0.4.6 to 0.4.7 (#4410)
  • 5cb16561 build(deps): downgrade github.com/daixiang0/gci from 0.13.0 to 0.12.3 (#4484)
  • 98f0a9ea chore: nancy
  • 51a963fa chore: simplify comparators (#4499)
  • 2417da15 feat: add copyloopvar linter (#4382)
  • b96ff83e feat: add intrange linter (#4378)
  • 0683d451 feat: add sort-order option (#4467)
  • eaafdf36 feat: add verify command (#4527)
  • 803970f0 feat: allow running only a specific linter without modifying the file configuration (#4438)

... (truncated)

Changelog

Sourced from github.com/golangci/golangci-lint's changelog.

v1.57.0

  1. New linters
  2. Updated linters
    • dupword: from 0.0.13 to 0.0.14
    • gci: from 0.12.1 to 0.12.3
    • ginkgolinter: from 0.15.2 to 0.16.1 (new option force-expect-to, validate-async-intervals, and forbid-spec-pollution)
    • go-critic: from 0.11.1 to 0.11.2
    • go-critic: support of enable-all and disable-all options
    • go-spancheck: from 0.5.2 to 0.5.3
    • gomodguard: from 1.3.0 to 1.3.1
    • govet: deprecation of check-shadowing ⚠️
    • govet: disable temporarily httpresponse because of a bug golang/go#66259
    • misspell: add extra-words
    • musttag: from 0.8.0 to 0.9.0
    • nakedret: from 2.0.2 to 2.0.4
    • paralleltest: from 1.0.9 to 1.0.10
    • perfsprint: from 0.6.0 to 0.7.1 (new option strconcat)
    • protogetter: from 0.3.4 to 0.3.5
    • revive: add exclude option
    • sloglint: from 0.4.0 to 0.5.0 (new option no-global)
    • staticcheck: from 0.4.6 to 0.4.7
    • testifylint: from 1.1.2 to 1.2.0 (new option bool-compare)
    • unconvert: to HEAD (new options fast-math and safe)
    • wrapcheck: from 2.8.1 to 2.8.3
    • Disable copyloopvar and intrange on Go < 1.22
  3. Enhancements
    • 🧩 New custom linters system https://golangci-lint.run/plugins/module-plugins/
    • 🎉 Allow running only a specific linter without modifying the file configuration (--enable-only)
    • Allow custom sort order for the reports (output.sort-order)
    • Automatically adjust the maximum concurrency to the container CPU quota if run.concurrency=0
    • Add verify command to check the configuration against the JSON Schema
    • Option to strictly follow Go generated file convention (issues.exclude-generated-strict)
    • Syntax to not override severity from linters (@linter)
    • Use severities from gosec
    • Create automatically directory related to output.formats.path
    • Use the first issue without inline on mergeLineIssues on multiple issues
  4. Misc.
    • ⚠️ Inactivate deprecated linters (deadcode, exhaustivestruct, golint, ifshort, interfacer, maligned, nosnakecase, scopelint, structcheck, varcheck)
    • ⚠️ Deprecated CLI flags have been removed (deprecated since 2018)
    • ⚠️ Move show-stats option from run to output configuration section
    • ⚠️ Replace run.skip-xxx options by issues.exclude-xxx options
    • ⚠️ Replace output.format by output.formats with a new file configuration syntax
    • Internal rewrite of the CLI
    • Improve 'no go files to analyze' message
    • Use GOTOOLCHAIN=auto inside the Docker images
  5. Documentation

... (truncated)

Commits
  • ddc703d fix: disable temporarily httpresponse from govet (#4528)
  • 1b6f006 dev: silence help on some commands usage on error (#4542)
  • eaafdf3 feat: add verify command (#4527)
  • 6709c97 dev: refactor .golangci.yml configuration and fix up nolintlint issues (#4537)
  • 02ea91d build(deps): bump github.com/alexkohler/nakedret/v2 from 2.0.2 to 2.0.4 (#4540)
  • 584520c build(deps): bump github.com/ghostiam/protogetter from 0.3.4 to 0.3.5 (#4539)
  • 485cb6a build(deps): bump github.com/ryancurrah/gomodguard from 1.3.0 to 1.3.1 (#4538)
  • 225bf17 dev: fix exclude-rules text
  • 3dbe882 govet: add a warning about the deprecation of check-shadowing (#4535)
  • cba35e1 feat: improve 'no go files to analyze' message (#4536)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.56.2 to 1.57.0.
- [Release notes](https://github.com/golangci/golangci-lint/releases)
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md)
- [Commits](golangci/golangci-lint@v1.56.2...v1.57.0)

---
updated-dependencies:
- dependency-name: github.com/golangci/golangci-lint
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Used by dependabot - identifies all PRs created by dependabot kind/misc Categorizes issue or PR as a miscellaneuous one. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. release-note-none Denotes a PR that doesnt merit a release note. labels Mar 20, 2024
@tekton-robot
Copy link
Contributor

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a tektoncd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@tekton-robot tekton-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 20, 2024
@vdemeester
Copy link
Member

/approve
/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 20, 2024
@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 20, 2024
@tekton-robot tekton-robot merged commit d416bcb into main Mar 20, 2024
7 of 8 checks passed
@dependabot dependabot bot deleted the dependabot/go_modules/tools/github.com/golangci/golangci-lint-1.57.0 branch March 20, 2024 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dependencies Used by dependabot - identifies all PRs created by dependabot kind/misc Categorizes issue or PR as a miscellaneuous one. lgtm Indicates that a PR is ready to be merged. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. release-note-none Denotes a PR that doesnt merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants