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

legacy/1.x: Fix 'git cherry-pick' merge error causing pipelineIndentationStyle setting to not be correctly wired up #1050

Merged
merged 1 commit into from
Oct 2, 2019

Conversation

bergmeister
Copy link
Contributor

@bergmeister bergmeister commented Oct 2, 2019

Fixes PowerShell/vscode-powershell#2214

The backport PR #1024 did a correct cherry-pick but git did not merge the code change correctly and undid a line that was added in an earlier PR.

Locally verified.

@bergmeister bergmeister requested a review from rjmholt as a code owner October 2, 2019 22:05
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

@MarkKharitonov
Copy link

When is it going to be released?

Copy link
Member

Likely early next week.

TylerLeonhardt pushed a commit that referenced this pull request Nov 15, 2019
…fault set of PSSA rules (#1102)

* Fix PipelineIndentation configuration issue (#1050)

* legacy/1.x: Update minimum PSSA version to 1.18.3 (#1052)

* Backport of #1101: Add PSAvoidAssignmentToAutomaticVariable to the default set of PSSA rules
@agressiv
Copy link

agressiv commented Dec 5, 2019

It's been over a month, this whitespace before/after brace problem is still there - any update when this will make it live?

@TylerLeonhardt
Copy link
Member

Apologies for the delay. There's one more PR we're trying to get in before we do the releases. It should happen in the next couple weeks.

TylerLeonhardt pushed a commit that referenced this pull request Jan 8, 2020
…t got broken in #1024 (#1142)

* Fix PipelineIndentation configuration issue (#1050)

* legacy/1.x: Update minimum PSSA version to 1.18.3 (#1052)

* Fix wiring of WhitespaceInsideBrace and WhitespaceAroundPipe

* revert regression from master

* add trailing comma

* revert PSAvoidUsingCmdletAliases change
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.

5 participants