Skip to content

[Babel 8] Parse TSHeritageBase.expression as an expression#17139

Merged
JLHwung merged 6 commits intobabel:mainfrom
JLHwung:ts-class-implements-expression
Feb 23, 2025
Merged

[Babel 8] Parse TSHeritageBase.expression as an expression#17139
JLHwung merged 6 commits intobabel:mainfrom
JLHwung:ts-class-implements-expression

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Feb 18, 2025

Q                       A
Fixed Issues? #16679
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link babel/website#3056
Any Dependency Changes?
License MIT

@JLHwung JLHwung added PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release PR: Needs Docs labels Feb 18, 2025
@@ -0,0 +1,2 @@
// Every extend element here is invalid
interface Invalid extends foo[bar], foo?.bar, foo(), import.meta {}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately the last example is currently accepted by Babel 7. In this PR we fix that for Babel 8.

@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Feb 18, 2025

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/58749

@JLHwung JLHwung marked this pull request as draft February 18, 2025 15:53
@JLHwung JLHwung marked this pull request as ready for review February 18, 2025 19:48

function tsIsEntityName(
node: N.Expression,
): node is N.MemberExpression | N.Identifier {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This assertion should be updated.

@JLHwung JLHwung merged commit 89226c9 into babel:main Feb 23, 2025
55 checks passed
@JLHwung JLHwung deleted the ts-class-implements-expression branch February 23, 2025 13:12
laine-hallot pushed a commit to laine-hallot/uwu-parser that referenced this pull request Mar 31, 2025
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label May 26, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants