◐ Shell
clean mode source ↗

Remove redundant whitespace between parameters with new option (disabled by default) in UseConsistentWhitespace by bergmeister · Pull Request #1392 · PowerShell/PSScriptAnalyzer

Skip to content

Navigation Menu

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@bergmeister

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:$ParameterValue syntax as some people might prefer 0 or 1 whitespace here, the rule leaves those cases completely untouched.

PR Checklist

@bergmeister bergmeister changed the title Remove redundant white space between parameters with new option (disabled by default) in UseConsistentWhitespace Remove redundant whitespace between parameters with new option (disabled by default) in UseConsistentWhitespace

Jan 5, 2020

rjmholt

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

@rjmholt Thanks for the review, I addressed all recommendations

rjmholt

Labels

2 participants

@bergmeister @rjmholt