-
Notifications
You must be signed in to change notification settings - Fork 142
CORTX-30083: Fix Codacy issues under "net" directory #1935
Conversation
retest this please - rerunning vm-deployment test |
This issue/pull request has been marked as |
Can one of the admins verify this patch? |
This issue/pull request has been marked as |
0bf2773
to
381ebfc
Compare
381ebfc
to
baed35d
Compare
- Fixing Codacy issues under net/test dir Signed-off-by: Vinoth.V <vinoth.v@seagate.com>
baed35d
to
028569b
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.
LGTM
Problem: Codacy error "Double quote to prevent globing and words splitting". Solution: We are putting the variable references in double quotes. Signed-off-by: Vinoth.V <vinoth.v@seagate.com>
Signed-off-by: Vinoth.V vinoth.v@seagate.com
Problem Statement
Codacy error "Double quote to prevent globing and words splitting".
Design
We are putting the variable references in double quotes.
Coding
Checklist for Author
Testing
Checklist for Author
Impact Analysis
Checklist for Author/Reviewer/GateKeeper
Review Checklist
Checklist for Author
Documentation
Checklist for Author