Skip to content

fix(37940): 'Move to new file' refactoring removes spaces between decorator and property modifiers #37959

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

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

a-tarasyuk
Copy link
Contributor

Fixes #37940

@andrewbranch
Copy link
Member

With your latest change, what happens to decorators on their own line?

@decorator // <-- is there a space here after a refactor?
class Foo {}

@a-tarasyuk
Copy link
Contributor Author

a-tarasyuk commented Apr 15, 2020

This change related only for the case when decorators are located on the single line

@decorator class Foo {}

The following case handled by

@decorator
class Foo {}

writeLine(closingLineTerminatorCount);

@andrewbranch
Copy link
Member

Thanks @a-tarasyuk! 🌟

@andrewbranch andrewbranch merged commit a16c441 into microsoft:master Apr 15, 2020
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.

'Move to new file' refactoring removes spaces between decorator and property modifiers
2 participants