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

Update typescript-estree to 5.x #11676

Merged
merged 10 commits into from
Oct 20, 2021

Conversation

sosukesuzuki
Copy link
Member

@sosukesuzuki sosukesuzuki commented Oct 13, 2021

Description

Closes #11668

Updates typescript-estree to 5.x.

Official release note

Notable changes:

  • Remove TSParenthesizedType node.
  • ClassProperty is now called PropertyDefinition
  • TSAbstractClassProperty is now called TSAbstractPropertyDefinition
  • TSAbstractPropertyDefinition.value is now always null
    • Values of abstract properties are removed from AST. So we should detect it and throw error like for invalid decorators.
  • Add full support for PrivateIdentifier
    • We can remove temporary workaround for TSPrivateIdentifier.

Checklist

Try the playground for this PR

@sosukesuzuki sosukesuzuki changed the title Support ts estree 5 Support typescript-estree 5 Oct 13, 2021
@sosukesuzuki sosukesuzuki force-pushed the support-ts-estree-5 branch 4 times, most recently from 0fd9a3e to 6d8cd94 Compare October 15, 2021 07:09
Comment on lines +101 to +113
=====================================input======================================
type Foo =
// prettier-ignore
(
aa
);

=====================================output=====================================
type Foo =
// prettier-ignore
aa;

================================================================================
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TSParenthesizedType node is removed, so we cannot keep parens wrap types with prettier-ignore typescript-eslint/typescript-eslint#3340

@sosukesuzuki sosukesuzuki changed the base branch from dependabot/npm_and_yarn/typescript-eslint/typescript-estree-5.0.0 to main October 19, 2021 06:42
@sosukesuzuki sosukesuzuki changed the title Support typescript-estree 5 Update typescript-estree to 5.x Oct 19, 2021
@sosukesuzuki sosukesuzuki marked this pull request as ready for review October 19, 2021 07:15
@@ -1,54 +1,18 @@
"use strict";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename as postprocess/index.js ?

Copy link
Member

@fisker fisker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

@sosukesuzuki sosukesuzuki merged commit 65e01b7 into prettier:main Oct 20, 2021
@sosukesuzuki sosukesuzuki deleted the support-ts-estree-5 branch October 20, 2021 06:13
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 21, 2022
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.

None yet

2 participants