-
-
Notifications
You must be signed in to change notification settings - Fork 238
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
fix: simplify variable initialization #1283
fix: simplify variable initialization #1283
Conversation
|
I try to undo the changes that my code formatter was done
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.
Don't delete this file, Please get it back
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.
remove changes to cli.js and as usual make sure you do not use any autoformatter
done |
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.
cli.js
is still showing up in the list of modified files
let me create another PR for this issue |
@thebedigupta no need to open a separate PR, by mistake, you've pushed the empty -> Copy and paste the content of the original cli.js file to your empty file. |
Oh it's that much easy ok I will do it and thank you for your suggestion |
74875d1
to
3a53813
Compare
Quality Gate passedIssues Measures |
I tried it but it was not working i don't know why anyway let me create another PR for removing these |
Description
In this PR I try to solve this issue
I am also attching test files captured results test-results.txt