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.

Sign up for GitHub

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

Jump to bottom

Squiz.WhiteSpace.ScopeClosingBraceclosure argument indenting incorrect with multi-line strings #1638

Closed
turbo8p opened this issue Sep 1, 2017 · 1 comment
Closed

Squiz.WhiteSpace.ScopeClosingBraceclosure argument indenting incorrect with multi-line strings #1638

turbo8p opened this issue Sep 1, 2017 · 1 comment
Labels
Type: bug
Milestone
3.1.0

Comments

@turbo8p
Copy link

turbo8p commented Sep 1, 2017 •
edited
Loading

Refer to issue #1637, the same problem still persist with
Squiz.WhiteSpace.ScopeClosingBrace.Indent
@turbo8p turbo8p mentioned this issue Sep 1, 2017
Closed
@gsherwood gsherwood changed the title Squiz.WhiteSpace.ScopeClosingBrace.Indent closure argument indenting incorrect with multi-line strings Squiz.WhiteSpace.ScopeClosingBraceclosure argument indenting incorrect with multi-line strings Sep 4, 2017
@gsherwood gsherwood added this to the 3.1.0 milestone Sep 4, 2017
gsherwood added a commit that referenced this issue Sep 4, 2017
@gsherwood
Fixed bug #1638 : Squiz.WhiteSpace.ScopeClosingBrace closure argument…
cd9c551 
… indenting incorrect with multi-line strings
@gsherwood
Copy link
Member

Same fix applied here.

@dependencies dependencies bot mentioned this issue Aug 7, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: bug
Projects
None yet
Milestone
3.1.0
Development

No branches or pull requests
2 participants
@gsherwood @turbo8p