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

Fix UseCorrectCasing to be actually configurable via"powershell.codeFormatting.useCorrectCasing" #910

Conversation

bergmeister
Copy link
Collaborator

@bergmeister bergmeister commented Apr 7, 2019

Make useCorrectCasing setting configurable by "powershell.codeFormatting.useCorrectCasing". It was correctly sent from the extension but rather hard-coded by accident in PSES, which made it not possible to turn the setting off.
This needs to be backported. cc @TylerLeonhardt

…orrectCasing" was not propagated correctly to PSES
@bergmeister bergmeister changed the title FIx UseCorrectCasing to be configurable via"powershell.codeFormatting.useCorrectCasing" Fix UseCorrectCasing to be actually configurable via"powershell.codeFormatting.useCorrectCasing" Apr 7, 2019
@TylerLeonhardt
Copy link
Member

TylerLeonhardt commented Apr 8, 2019

@bergmeister we're also defaulting this to False because of the known issue with the rule changing git to git.exe etc, right? it's in the vscode-powershell PR.

global.json Outdated Show resolved Hide resolved
Copy link
Member

@TylerLeonhardt TylerLeonhardt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TylerLeonhardt TylerLeonhardt merged commit 199cf55 into PowerShell:master Apr 8, 2019
bergmeister added a commit to bergmeister/PowerShellEditorServices that referenced this pull request Apr 9, 2019
…ormatting.useCorrectCasing" (PowerShell#910)

* Make UseCorrectCasing configurable as "powershell.codeFormatting.useCorrectCasing" was not propagated correctly to PSES

* re-trigger ci

* remove global.json
rjmholt pushed a commit to rjmholt/PowerShellEditorServices that referenced this pull request Apr 10, 2019
…ormatting.useCorrectCasing" (PowerShell#910)

* Make UseCorrectCasing configurable as "powershell.codeFormatting.useCorrectCasing" was not propagated correctly to PSES

* re-trigger ci

* remove global.json
rjmholt pushed a commit that referenced this pull request Apr 10, 2019
…ormatting.useCorrectCasing" (#910)

* Make UseCorrectCasing configurable as "powershell.codeFormatting.useCorrectCasing" was not propagated correctly to PSES

* re-trigger ci

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

Successfully merging this pull request may close these issues.

None yet

3 participants