Skip to content

[0.76] Skip hermes-parser under Babel for non-Flow JS code#47569

Merged
blakef merged 1 commit intofacebook:0.76-stablefrom
huntie:0.76-fix-babel-parsing
Nov 13, 2024
Merged

[0.76] Skip hermes-parser under Babel for non-Flow JS code#47569
blakef merged 1 commit intofacebook:0.76-stablefrom
huntie:0.76-fix-babel-parsing

Conversation

@huntie
Copy link
Copy Markdown
Member

@huntie huntie commented Nov 12, 2024

Summary:

Resolves facebook/hermes#1549.

Changelog:
[General][Fixed] - When using Babel with plain JavaScript files, support for additional user syntax plugins should be fixed (now uses Babel's parser instead of hermes-parser). There is no change for JS files annotated with @flow, where extended JS syntax remains unsupported.

Test Plan

[Meta only] See Phabricator test plan attached to facebook/hermes#1556.

See also hermes-parser changelog — no anticipated incompatibilities from single package bump.

image

@huntie huntie requested review from blakef and cipolleschi November 12, 2024 11:59
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants