-
-
Notifications
You must be signed in to change notification settings - Fork 100
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: dead code and linter updates #333
Conversation
nandor-magyar
commented
Nov 11, 2022
- new golangci lint rules & fixes
- removed a bunch of dead code
- removed unused variables
30a9e10
to
206db53
Compare
Codecov ReportBase: 18.29% // Head: 18.50% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## develop #333 +/- ##
===========================================
+ Coverage 18.29% 18.50% +0.21%
===========================================
Files 53 52 -1
Lines 5077 5019 -58
===========================================
Hits 929 929
+ Misses 4125 4068 -57
+ Partials 23 22 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
6a82138
to
baf43eb
Compare
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.
.
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.
🎬
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
baf43eb
to
445f324
Compare
refactor(crux-ui): Separate contains methods from image config fix(crux): Add common error interceptor fix(crux-ui): Node status indicator position center feat(crux-ui): Add back btn to image details fix(crux-ui): Image list actions responsive display fix(crux-ui): Image config sections improvements fix(crux-ui): Deployment delete navigate fix(crux-ui): Confirmation modal cancel fix fix(agent): Inject secret key Co-Authored-By: Nandor Magyar <nandormagyar.it@gmail.com> fix(crux): Node create get identity notification error
445f324
to
83f10f1
Compare