Skip to content
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

UseConsistentWhitespace: Fix CheckParameter bug when using interpolated string #1498

Merged
merged 1 commit into from
May 18, 2020

Conversation

bergmeister
Copy link
Collaborator

@bergmeister bergmeister commented May 14, 2020

PR Summary

Fixes #1495

I thought the searchNestedScriptBlocks: false option would not make it return nested objects in commandAst.FindAll. However, for an interpolated string, the VariableExpressionAst of the ExpendableStringExpressionAst is returned as well, therefore checking that only AST are returned whose parent is the commandAst.

PR Checklist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CheckParameter option of UseConsistentWhitespace (off by default): When turned, on removes part of script
2 participants