-
Notifications
You must be signed in to change notification settings - Fork 142
CORTX-30751: Codacy code cleanup (#1606) #1923
Conversation
Can one of the admins verify this patch? |
This issue/pull request has been marked as |
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.
wrong.
This issue/pull request has been marked as |
This patch fixes some of the codacy warnings. warning fixed : "Double quote to prevent globing and words splitting". Signed-off-by: alfhad <fahadshah2411@gmail.com>
Jenkins CI Result : Motr#1619Motr Test Summary
CppCheck SummaryCppcheck: No new warnings found 👍 |
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
ok to test |
1 similar comment
ok to test |
This patch fixes some of the codacy warnings. warning fixed : "Double quote to prevent globing and words splitting".
This patch fixes some of the codacy warnings.
warning fixed : "Double quote to prevent globing and words splitting".
Signed-off-by: alfhad fahadshah2411@gmail.com
Problem Statement
We see 1688 occurrence of pattern, "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