Skip to content

Update module golangci/golangci-lint to v1.19.0 #10207

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

Closed

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Sep 23, 2019

This PR contains the following updates:

Package Type Update Change
github.com/golangci/golangci-lint require minor v1.18.0 -> v1.19.0

Release Notes

golangci/golangci-lint

v1.19.0

Compare Source

Changelog

New Linters

92168d0 Add dogsled linter. (#​705)
c7dee2c Add whitespace linter (#​673)
5884581 linters: add Godox linter support (#​621)

Linters Updates

6a979fb Update staticcheck and cache go/analysis facts (#​699)
9976bdc Update whitespace
94eaa8f update go-critic (#​679)
bf67a3e update gosec (#​681)
b083358 update scopelint (#​680)
9161de5 Upgrade depguard to fix performance issue with go modules (#​698)

Performance Optimization

2fc8b7b Drop memory usage of go/analysis linters 5x
3aade55 Reduce memory usage of go/analysis
c9a9255 Speed up packages loading (#​693)
fc8d614 speedup skip-dirs processing by caching

Other

f1c1dbf Feature/enable autofix on whitespace (#​674)
f3f5dfc govet: implement analyzers config (#​697)
c215cff Add bash completion to homebrew formula (#​676)
4e439b4 Fix #​608: use forked bodyclose
4495f89 Log go/analysis panics, don't crash
6ec8ff2 Update Dockerfile to Go 1.13. (#​695)

Docker images

  • docker pull golangci/golangci-lint:latest
  • docker pull golangci/golangci-lint:v1.19.0
  • docker pull golangci/golangci-lint:v1.19

Renovate configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot. View repository job log here.

@renovate renovate bot requested a review from a team September 23, 2019 18:03
@renovate renovate bot added the dependencies Used to indicate dependency changes. label Sep 23, 2019
@ghost ghost added the size/XXL Managed by automation to categorize the size of a PR. label Sep 23, 2019
@renovate renovate bot force-pushed the renovate/git.colasdn.top-golangci-golangci-lint-1.x branch 2 times, most recently from fd6e51f to b07aefa Compare September 23, 2019 19:51
@renovate renovate bot changed the title chore(deps): update module golangci/golangci-lint to v1.18.0 chore(deps): update module golangci/golangci-lint to v1.19.0 Sep 23, 2019
@renovate
Copy link
Contributor Author

renovate bot commented Sep 23, 2019

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻️ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you check the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: go.sum
Command failed: docker run --rm -v /mnt/renovate/gh/terraform-providers/terraform-provider-aws:/mnt/renovate/gh/terraform-providers/terraform-provider-aws -v /tmp/renovate-cache/others/go:/tmp/renovate-cache/others/go -e GOPATH -e GOPROXY -e CGO_ENABLED=0 -w /mnt/renovate/gh/terraform-providers/terraform-provider-aws renovate/go bash -c "git config --global url.\"https://x-access-token:**redacted**@github.com/\".insteadOf \"https://github.com/\" && go get -d ./..."
go: github.com/golangci/[email protected] requires
	github.com/goreleaser/[email protected] requires
	code.gitea.io/[email protected] requires
	github.com/go-macaron/[email protected]: invalid pseudo-version: revision is longer than canonical (6fd6a9bfe14e)

@ghost ghost added size/XS Managed by automation to categorize the size of a PR. and removed size/XXL Managed by automation to categorize the size of a PR. labels Sep 23, 2019
@renovate renovate bot force-pushed the renovate/git.colasdn.top-golangci-golangci-lint-1.x branch from b07aefa to 1c66930 Compare September 23, 2019 21:03
@renovate renovate bot force-pushed the renovate/git.colasdn.top-golangci-golangci-lint-1.x branch from 1c66930 to 0bc9501 Compare September 23, 2019 23:21
@bflad
Copy link
Contributor

bflad commented Sep 23, 2019

This is pending upstream fixes: golangci/golangci-lint#730

@bflad bflad added the upstream Addresses functionality related to the cloud provider. label Sep 23, 2019
@renovate renovate bot changed the title chore(deps): update module golangci/golangci-lint to v1.19.0 Update module golangci/golangci-lint to v1.19.0 Sep 24, 2019
@bflad
Copy link
Contributor

bflad commented Sep 24, 2019

Closing to pick up golangci/golangci-lint@c427c61 or the next release

@bflad bflad closed this Sep 24, 2019
@renovate
Copy link
Contributor Author

renovate bot commented Sep 24, 2019

Renovate Ignore Notification

As this PR has been closed unmerged, Renovate will now ignore this update (v1.19.0). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.

If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.

@renovate renovate bot deleted the renovate/git.colasdn.top-golangci-golangci-lint-1.x branch September 24, 2019 20:12
@ghost
Copy link

ghost commented Nov 1, 2019

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 and limited conversation to collaborators Nov 1, 2019
@breathingdust breathingdust added the linter Pertains to changes to or issues with the various linters. label Oct 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Used to indicate dependency changes. linter Pertains to changes to or issues with the various linters. size/XS Managed by automation to categorize the size of a PR. upstream Addresses functionality related to the cloud provider.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants