Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Pin golangci-lint version #67

Merged
merged 2 commits into from
Aug 23, 2024
Merged

Pin golangci-lint version #67

merged 2 commits into from
Aug 23, 2024

Conversation

ldemailly
Copy link
Member

@ldemailly ldemailly commented Aug 23, 2024

Pending golangci/golangci-lint-action#1091

Will avoid future breakages without code change or go version changes like we had with gosec int conversions

ldemailly added a commit to grol-io/grol that referenced this pull request Aug 23, 2024
Same as fortio/workflows#67
(need to figure cross org share actually rather than copy-pasta)
@ldemailly ldemailly merged commit 3c210bc into main Aug 23, 2024
5 checks passed
@ldemailly ldemailly deleted the ldemailly-patch-2 branch August 23, 2024 00:41
@ldemailly ldemailly restored the ldemailly-patch-2 branch August 23, 2024 00:41
@ldemailly
Copy link
Member Author

ldemailly commented Aug 23, 2024

nvm this one is good it's the grol one that's bad

@ldemailly ldemailly deleted the ldemailly-patch-2 branch August 23, 2024 00:43
ldemailly added a commit to grol-io/grol that referenced this pull request Aug 23, 2024
* Pin linter version

Same as fortio/workflows#67
(need to figure cross org share actually rather than copy-pasta)

* Fix the nolintlint now that gosec is fixed and the fix in linters 1.60.3

* Update .github/workflows/gochecks.yml
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants