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

Fix Flight Prod Fixture #20382

Merged
merged 3 commits into from
Dec 5, 2020
Merged

Conversation

sebmarkbage
Copy link
Collaborator

Something is messed with Babel where it inserts a dependency on an async helper which is an ESM module. This doesn't work in modern Node from a CommonJS file so it errors before it has a chance to transpile it. I have no idea why this is only affecting prod.

The solution is just to not use async/await.

Also wired up some helper so now the prod example works.

Babel transpilation fails for some reason in prod.
Uses python because meh. Just to show it's static.
@facebook-github-bot facebook-github-bot added React Core Team Opened by a member of the React Core Team CLA Signed labels Dec 5, 2020
@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 5, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d96f5f3:

Sandbox Source
React Configuration

@sizebot
Copy link

sizebot commented Dec 5, 2020

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against d96f5f3

@sizebot
Copy link

sizebot commented Dec 5, 2020

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against d96f5f3

@sebmarkbage sebmarkbage merged commit 0db61a0 into facebook:master Dec 5, 2020
koto pushed a commit to koto/react that referenced this pull request Jun 15, 2021
* Don't use async/await

Babel transpilation fails for some reason in prod.

* Set up production runner command

Uses python because meh. Just to show it's static.

* Use build folder in prod
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants