-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
chore: enable use-any from revive #21282
chore: enable use-any from revive #21282
Conversation
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #21282 +/- ##
==========================================
- Coverage 55.19% 53.28% -1.92%
==========================================
Files 337 337
Lines 57058 57058
==========================================
- Hits 31496 30403 -1093
- Misses 22863 24000 +1137
+ Partials 2699 2655 -44 ☔ View full report in Codecov by Sentry. |
61ed376
to
4972abf
Compare
f09552b
to
3f69bdd
Compare
3f69bdd
to
7864b37
Compare
Signed-off-by: Matthieu MOREL <[email protected]>
7864b37
to
9e041e4
Compare
Idk, I kinda like |
I'm fine either way. I am curious why the changes to openapi_generated.go aren't causing CI failures. |
Figured that out: #21350 I like |
Description
revive helps ensure a number of rules
use-any proposes to replace instances of interface{} with any.
By default revive has a list of rules enabled. This includes this list and disables those that are not yet applied.