-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - Remove dead code #1503
Conversation
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
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: Burak Sekili <buraksekili@gmail.com>
5efdca8
488d14d
to
5efdca8
Compare
Codecov Report
@@ Coverage Diff @@
## master #1503 +/- ##
=======================================
Coverage 68.24% 68.24%
=======================================
Files 132 132
Lines 10747 10747
=======================================
Hits 7334 7334
Misses 2826 2826
Partials 587 587
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: Burak Sekili <buraksekili@gmail.com>
Signed-off-by: Burak Sekili <buraksekili@gmail.com> Signed-off-by: skovacevic <stefan.kovacevic@mainflux.io>
Signed-off-by: Burak Sekili buraksekili@gmail.com
What does this do?
This PR removes a dead (commented) code as described here: https://github.com/mainflux/mainflux/pull/1502#discussion_r753387842
Which issue(s) does this PR fix/relate to?
-
List any changes that modify/break current functionality
-
Have you included tests for your changes?
-
Did you document any new/modified functionality?
-
Notes