Skip to content

add beautifier rule for space after close paren and destructure bracket#21859

Merged
mhegazy merged 1 commit into
microsoft:masterfrom
grubeeric:beautifier_destructure_rule
Feb 10, 2018
Merged

add beautifier rule for space after close paren and destructure bracket#21859
mhegazy merged 1 commit into
microsoft:masterfrom
grubeeric:beautifier_destructure_rule

Conversation

@grubeeric
Copy link
Copy Markdown
Contributor

Fixes #21675

Adds a new rule to the javascript beautifier to add a space after a closing parenthesis if the next token is an open bracket and the context is a control declaration.

Tests added as well.

@msftclas
Copy link
Copy Markdown

msftclas commented Feb 10, 2018

CLA assistant check
All CLA requirements met.

@grubeeric grubeeric force-pushed the beautifier_destructure_rule branch from f933dae to 70d0409 Compare February 10, 2018 14:42
@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Feb 10, 2018

Thanks!

@mhegazy mhegazy merged commit 879cb69 into microsoft:master Feb 10, 2018
@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Feb 10, 2018

Some more formatting issues if you are still interested in the area

@microsoft microsoft locked and limited conversation to collaborators Jul 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants