-
Notifications
You must be signed in to change notification settings - Fork 68
🌱 Upgrade golangci-lint from v2.1.6 to v2.6.2 #2336
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
🌱 Upgrade golangci-lint from v2.1.6 to v2.6.2 #2336
Conversation
**Steps Executed** - with go 1.24.6 - bingo get golangci-lint@none - bingo get github.com/golangci/golangci-lint/v2/cmd/golangci-lint@latest
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
… require.ErrorAs depending on the case
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.
Pull Request Overview
This PR upgrades golangci-lint from v2.1.6 to v2.6.2, which includes updates to numerous linter dependencies and improvements to the bingo build system.
Key Changes:
- Upgraded golangci-lint from v2.1.6 to v2.6.2
- Updated hundreds of linter dependencies to their latest versions
- Enhanced bingo build system with cross-compilation support for host platform
- Updated bingo tooling from v0.9 to v0.10
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .bingo/variables.env | Updated golangci-lint version reference and bingo version to v0.10 |
| .bingo/golangci-lint.sum | Updated all transitive dependencies for golangci-lint v2.6.2 |
| .bingo/golangci-lint.mod | Updated golangci-lint version and Go version requirement |
| .bingo/Variables.mk | Added GOHOSTOS/GOHOSTARCH/GOHOSTARM support for cross-compilation and updated version references |
| .bingo/README.md | Improved documentation formatting and updated Go version requirements |
| .bingo/.gitignore | Added *tmp.sum to ignore list |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: perdasilva 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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2336 +/- ##
=======================================
Coverage 74.30% 74.30%
=======================================
Files 91 91
Lines 7083 7083
=======================================
Hits 5263 5263
Misses 1405 1405
Partials 415 415
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e20c9bc
into
operator-framework:main
Steps Executed
Description
Reviewer Checklist