Skip to content

Fix(parser): flow parser small fixes#17865

Merged
nicolo-ribaudo merged 6 commits into
babel:mainfrom
JLHwung:fix-flow-parser
Mar 25, 2026
Merged

Fix(parser): flow parser small fixes#17865
nicolo-ribaudo merged 6 commits into
babel:mainfrom
JLHwung:fix-flow-parser

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Mar 11, 2026

Q                       A
Fixed Issues? Fixes #1, Fixes #2
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This PR includes commits from #17853, please review that PR first.

In this PR we fixed a few @ts-expect-error comments and aligned the parser implementation to the current flowjs parser and the type definitions. Each fix is a separate commit so please review it by commits.

@JLHwung JLHwung added PR: Spec Compliance 👓 A type of pull request used for our changelog categories area: flow labels Mar 11, 2026
@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Mar 11, 2026

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

@JLHwung JLHwung marked this pull request as ready for review March 11, 2026 15:01
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 11, 2026

Open in StackBlitz

commit: 6a869b9

@nicolo-ribaudo nicolo-ribaudo merged commit 25432ba into babel:main Mar 25, 2026
56 checks passed
@nicolo-ribaudo nicolo-ribaudo deleted the fix-flow-parser branch March 25, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: flow pkg: parser PR: Spec Compliance 👓 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants