-
Notifications
You must be signed in to change notification settings - Fork 108
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
Address info level code smells #2361
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2361 +/- ##
=========================================
Coverage 84.38% 84.39%
+ Complexity 2315 2308 -7
=========================================
Files 440 439 -1
Lines 12004 11977 -27
Branches 1022 1020 -2
=========================================
- Hits 10130 10108 -22
+ Misses 1556 1551 -5
Partials 318 318 Continue to review full report at Codecov.
|
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
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
e27053a
to
edc8456
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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
* Fix info level code smells * Remove unused REST API performance tests * Remove unused TimestampConverter Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Description:
TimestampConverter
Related issue(s):
Fixes #2336
Notes for reviewer:
There was one info code smell for perf test. Instead of fixing I just removed the whole folder 😄 . But we don't use the REST API performance and we rely on k6 tests now anyway.
Checklist