Fix bug with PipelineIndentationStyle.None where break instead of continue statement was used by bergmeister · Pull Request #1497 · PowerShell/PSScriptAnalyzer
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh! My switch suggestion got lost! Ugh, too much context switching... Anyway, not an issue
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought I addressed your comment here by not initializing it: #1497 (comment)
Or did you mean a switch around tokens[searchIndex].Kind? I don't see too much value in it tbh. Switch statements don't look that much cleaner for simple cases like that IMHO because of their ceremony that requires additional indentation layer, break statements, etc..
Otherwise we can always create a follow up PR