-
Notifications
You must be signed in to change notification settings - Fork 115
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 code smells #9910
Fix code smells #9910
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9910 +/- ##
============================================
- Coverage 92.25% 92.25% -0.01%
Complexity 7821 7821
============================================
Files 953 953
Lines 32731 32730 -1
Branches 4146 4146
============================================
- Hits 30196 30195 -1
Misses 1559 1559
Partials 976 976 ☔ View full report in Codecov by Sentry. |
Signed-off-by: sdimitrov9 <[email protected]>
8481a02
to
51fdb80
Compare
...-mirror-grpc/src/test/java/com/hedera/mirror/grpc/repository/TopicMessageRepositoryTest.java
Show resolved
Hide resolved
…ew other sonar issues Signed-off-by: sdimitrov9 <[email protected]>
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.
LGTM
Looks like your change violates spotless check |
8ab4217
to
83ec3b6
Compare
Signed-off-by: sdimitrov9 <[email protected]>
83ec3b6
to
4e064e2
Compare
|
Fixed + additional code smells fixes added to the latest commit |
Description:
200+ sonar issues are expected to be fixed with this PR
Related issue(s):
Partially fixes 9010
Checklist