- Notifications
You must be signed in to change notification settings - Fork407
Remove redundant whitespace between parameters with new option (disabled by default) in UseConsistentWhitespace#1392
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Co-Authored-By: Robert Holt <rjmholt@gmail.com>
…ghtExtent variable, which broke build.Variable is used 2 times and makes code more readable, therefore keeping it
bergmeister commentedJan 6, 2020
@rjmholt Thanks for the review, I addressed all recommendations |
PR Summary
Disabled by default because it is new. The plan is to later add an option to the vs code extension and pending positive feedback we can then later enable it by default.
The only case that was excluded by design is when the user uses the
-ParameterName:$ParameterValuesyntax as some people might prefer 0 or 1 whitespace here, the rule leaves those cases completely untouched.PR Checklist
.cs,.ps1and.psm1files have the correct copyright headerWIP:to the beginning of the title and remove the prefix when the PR is ready.