-
Notifications
You must be signed in to change notification settings - Fork 592
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
fix(deps): update module github.com/golangci/golangci-lint to v1.64.2 #6758
fix(deps): update module github.com/golangci/golangci-lint to v1.64.2 #6758
Conversation
ℹ Artifact update noticeFile name: tools/go.modIn order to perform the update(s) described in the table above, Renovate ran the
Details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6758 +/- ##
=====================================
Coverage 75.1% 75.1%
=====================================
Files 203 203
Lines 18323 18323
=====================================
+ Hits 13769 13775 +6
+ Misses 4144 4140 -4
+ Partials 410 408 -2
|
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
@@ -63,7 +63,7 @@ func validateReturns(handler reflect.Type) error { | |||
} | |||
|
|||
// Wraps and calls customer lambda handler then unpacks response as necessary. | |||
func (whf *wrappedHandlerFunction) wrapperInternals(ctx context.Context, handlerFunc interface{}, eventJSON []byte, event reflect.Value, takesContext bool) (interface{}, error) { | |||
func (whf *wrappedHandlerFunction) wrapperInternals(ctx context.Context, handlerFunc interface{}, eventJSON []byte, event reflect.Value, takesContext bool) (interface{}, error) { // nolint:revive // this control flag is required |
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.
This lint issue is surprising, and maybe a false positive?
flag-parameter: parameter 'len' seems to be a control flag, avoid control coupling
There is no len
parameter though.
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.
Looks like already fixed but not released: mgechev/revive#1211
cc @open-telemetry/go-approvers since I had to add a commit. |
This PR contains the following updates:
v1.63.4
->v1.64.2
Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
golangci/golangci-lint (github.com/golangci/golangci-lint)
v1.64.2
Compare Source
v1.64.1
Compare Source
v1.64.0
Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.