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

Revert "[TypeScript] format TSAsExpression with same logic as BinaryExpression" #7958

Merged
merged 3 commits into from Apr 6, 2020

Conversation

thorn0
Copy link
Member

@thorn0 thorn0 commented Apr 6, 2020

Reverts #7869

fixes #7956
fixes #7957
reopens #6443

@sosukesuzuki
Copy link
Member

sosukesuzuki commented Apr 6, 2020

@thorn0 I'm sorry that the PR I created caused a regression. Thanks for the revert.

I was surprised that 2.0.3 was released suddenly. Could you tell me about the release cycle? Even if it was a patch version release, we should have used other source code to check for regressions.

@thorn0
Copy link
Member Author

thorn0 commented Apr 6, 2020

@sosukesuzuki The new major version of the VS Code extension was broken because of #7949 That's why I made this patch release. But I should've tested it more thoroughly, indeed. :(

@thorn0 thorn0 merged commit 50335bf into master Apr 6, 2020
@thorn0
Copy link
Member Author

thorn0 commented Apr 6, 2020

Hopefully, we're done with 2.0 regressions and there will be no more 2.0.x releases.

@alexander-akait alexander-akait deleted the revert-7869-tsasexp branch April 6, 2020 13:22
@wincent
Copy link

wincent commented Apr 6, 2020

Thanks for the quick response on this one @thorn0.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 7, 2021
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.

TypeScript as regression (2.0.3) Regression with TypeScript "as const" object literal in 2.0.3
4 participants