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

build(deps): bump github.com/golangci/golangci-lint from 1.35.2 to 1.37.1 in /tools #17743

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 22, 2021

Bumps github.com/golangci/golangci-lint from 1.35.2 to 1.37.1.

Release notes

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

v1.37.1

Changelog

9c477156 Update doc for v1.37.0 (#1742) 32e8517a cyclop: add missing settings (#1743) b39dbcd6 durationcheck: False positive when multiplying with int type struct field (#1744) ea5f4790 exhaustivestruct: add missing settings (#1746) 4b3fb4c6 revive: fix types and default configuration. (#1747)

v1.37.0

Changelog

89367aee Add "exclude" option for godot linter (#1669) 35b6f354 Add "revive" linter (#1729) 123da8e6 Add go1.16 support (#1740) 3ef13a80 Add plugin option for Goland (#1720) 6038573c Compile binaries with -trimpath enabled (#1688) 9e0c9f39 Expose more config options for forbidigo (#1677) 443e5b6c Print error message and exit with non-zero status when ruleguard parse error occurs (#1666) af6842c5 Remove gometalinter comparison (#1731) 5d75889c Update docs for v1.36.0 (#1668) 0a474a26 Update ifshort to v1.0.1 (#1671) 34e5fc63 Use upstream gocyclo. (#1739) 2121370f add 'cyclop' linter (#1738) 7dc0f311 build(deps): bump actions/cache from v2 to v2.1.4 (#1721) cb028491 build(deps): bump gatsby from 2.29.3 to 2.32.3 in /docs (#1716) 683dbea7 build(deps): bump gatsby-plugin-canonical-urls in /docs (#1713) db8ebb2b build(deps): bump gatsby-plugin-catch-links in /docs (#1710) 28058d61 build(deps): bump gatsby-plugin-google-analytics in /docs (#1715) 2ec034da build(deps): bump gatsby-plugin-manifest from 2.9.1 to 2.12.0 in /docs (#1704) 32c44df1 build(deps): bump gatsby-plugin-mdx from 1.7.1 to 1.10.0 in /docs (#1706) c3eb769c build(deps): bump gatsby-plugin-netlify from 2.8.0 to 2.11.0 in /docs (#1700) ea76542d build(deps): bump gatsby-plugin-offline from 3.7.1 to 3.10.0 in /docs (#1701) dfa7ab59 build(deps): bump gatsby-plugin-react-helmet in /docs (#1709) 2982b131 build(deps): bump gatsby-plugin-sharp from 2.11.2 to 2.14.1 in /docs (#1699) eb326949 build(deps): bump gatsby-plugin-sitemap from 2.9.0 to 2.12.0 in /docs (#1703) 32487526 build(deps): bump gatsby-remark-autolink-headers in /docs (#1674) 8ff2b391 build(deps): bump gatsby-remark-autolink-headers in /docs (#1711) 2c1dbb9a build(deps): bump gatsby-remark-copy-linked-files in /docs (#1705) 8deb0ca6 build(deps): bump gatsby-remark-images from 3.8.1 to 3.11.0 in /docs (#1702) 3505128b build(deps): bump gatsby-remark-responsive-iframe in /docs (#1698) 3790d440 build(deps): bump gatsby-source-filesystem from 2.8.1 to 2.11.0 in /docs (#1714) aafdee85 build(deps): bump gatsby-transformer-remark in /docs (#1719) 6c1c9ccb build(deps): bump gatsby-transformer-sharp from 2.9.0 to 2.11.0 in /docs (#1673) 8bfdb1ff build(deps): bump gatsby-transformer-sharp in /docs (#1712) 20402078 build(deps): bump gatsby-transformer-yaml from 2.8.0 to 2.11.0 in /docs (#1718) 3096f8ae build(deps): bump github.com/ashanbrown/forbidigo from 1.0.0 to 1.1.0 (#1678) ed9480c4 build(deps): bump github.com/kulti/thelper from 0.3.0 to 0.3.1 (#1722) 5694c501 build(deps): bump github.com/mbilski/exhaustivestruct from 1.1.0 to 1.2.0 (#1680) 4dc0dc32 build(deps): bump github.com/spf13/cobra from 1.1.1 to 1.1.3 (#1736)

... (truncated)

Commits

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 dependabot bot requested a review from a team as a code owner February 22, 2021 07:29
@dependabot dependabot bot added dependencies Used to indicate dependency changes. go labels Feb 22, 2021
@ghost ghost added size/L Managed by automation to categorize the size of a PR. labels Feb 22, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 24, 2021

Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update by commenting @dependabot rebase.

bflad added a commit that referenced this pull request Feb 24, 2021
… impi handling in tools Go module, and update golangci-lint@v1.37.1

Reference: #17743
Reference: #17685
Reference: https://golang.org/doc/go1.16

This change request bundles a few related items around the Go 1.16 upgrade and changes to the `tools` Go module:

* Removes `GO111MODULE=on` environment variable handling in GitHub Actions and GNUmakefile as that is now the default behavior in Go 1.16
* Adds missing `impi` into `tools/main.go` so it shows as needing to be versions
* Upgrades `golangci-lint` in `tools` since dependabot has been having trouble with this change for some reason
* Runs `go mod tidy` in `tools` with Go 1.16, which is the majority of the automatic updates in `tools/go.sum`

Now with Go 1.16, we can use `go install PACKAGE@VERSION` if we no longer want to use the `tools` Go module strategy, however that will also remove dependabot pinging us on tooling updates.
@bflad
Copy link
Contributor

bflad commented Feb 24, 2021

Superseded by #17795

@bflad bflad closed this Feb 24, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 24, 2021

OK, I won't notify you again about this release, but will get in touch when a new version is available.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot dependabot bot deleted the dependabot/go_modules/tools/github.com/golangci/golangci-lint-1.37.1 branch February 24, 2021 17:54
@bflad
Copy link
Contributor

bflad commented Feb 25, 2021

Looks like the trouble here was similar to dependabot/dependabot-core#3073

bflad added a commit that referenced this pull request Feb 25, 2021
… impi handling in tools Go module, and update golangci-lint@v1.37.1 (#17795)

Reference: #17743
Reference: #17685
Reference: https://golang.org/doc/go1.16

This change request bundles a few related items around the Go 1.16 upgrade and changes to the `tools` Go module:

* Removes `GO111MODULE=on` environment variable handling in GitHub Actions and GNUmakefile as that is now the default behavior in Go 1.16
* Adds missing `impi` into `tools/main.go` so it shows as needing to be versions
* Upgrades `golangci-lint` in `tools` since dependabot has been having trouble with this change for some reason
* Runs `go mod tidy` in `tools` with Go 1.16, which is the majority of the automatic updates in `tools/go.sum`

Now with Go 1.16, we can use `go install PACKAGE@VERSION` if we no longer want to use the `tools` Go module strategy, however that will also remove dependabot pinging us on tooling updates.
@ghost
Copy link

ghost commented Mar 27, 2021

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Mar 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Used to indicate dependency changes. size/L Managed by automation to categorize the size of a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant