-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Modify golangci configuration to make it work. #8695
Conversation
c14afb6
to
b14480b
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8695 +/- ##
==========================================
- Coverage 59.48% 59.46% -0.02%
==========================================
Files 371 371
Lines 40191 40191
==========================================
- Hits 23909 23901 -8
- Misses 14789 14796 +7
- Partials 1493 1494 +1 ☔ View full report in Codecov by Sentry. |
For the record I proposed #8641 on 22 January. |
@mmorel-35 |
That's fine for me |
Signed-off-by: Xun Jiang <[email protected]>
b14480b
to
e736ef7
Compare
Now that this has been merged to main branch, I just rebased #8641 |
Thank you for contributing to Velero!
Please add a summary of your change
https://github.com/vmware-tanzu/velero/actions/runs/13363546193/job/37317100078#step:4:30
The linter check action failed after some linter PR merged.
This PR aims to resolve the failures.
Does your change fix a particular issue?
Fixes #(issue)
Please indicate you've done the following:
make new-changelog
) or comment/kind changelog-not-required
on this PR.site/content/docs/main
.