-
Notifications
You must be signed in to change notification settings - Fork 2.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
Enable Lint Rules: confusing-results & receiver-naming #5524
Conversation
Signed-off-by: FlamingSaint <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5524 +/- ##
===========================================
+ Coverage 54.24% 96.19% +41.94%
===========================================
Files 161 327 +166
Lines 8369 16013 +7644
===========================================
+ Hits 4540 15403 +10863
+ Misses 3383 435 -2948
+ Partials 446 175 -271
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: FlamingSaint <[email protected]>
hmm, DCO check is stuck. I see you changed your name/email in git, did you also change it the same way in your GitHub profile? |
I didn't change my name/email in git |
Which problem is this PR solving?
Description of the changes
How was this change tested?
make lint
make test
Checklist
jaeger
:make lint test
jaeger-ui
:yarn lint
andyarn test