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

[fbjs][fbjs-scripts] Upgrade core-js version #370

Merged
merged 1 commit into from
Apr 6, 2020
Merged

[fbjs][fbjs-scripts] Upgrade core-js version #370

merged 1 commit into from
Apr 6, 2020

Conversation

bigfootjon
Copy link
Member

Every time I use yarn I get a warning about how core-js < 3 is not supported for relay and flux through fbjs. This migrates core-js to version 3 for fbjs and fbjs-scripts.

@zpao please, you're my only hope

@bigfootjon
Copy link
Member Author

I force pushed to try and get TravisCI to run. It ran, and almost everything passed but the Node10 Lint job failed to download Node.JS, some how. Everything else passed!

@zpao zpao merged commit 4bd92de into facebook:master Apr 6, 2020
@bigfootjon bigfootjon deleted the upgrade-core-js branch April 6, 2020 17:31
@jdeff jdeff mentioned this pull request Jun 5, 2020
@polarathene
Copy link

polarathene commented Jun 22, 2020

@zpao while this has been merged, it's still 1.0.0 version in the package, no new release has been published to npm(last release 2 years ago).

The warning will continue as a result.

facebook-github-bot pushed a commit to facebook/relay that referenced this pull request Sep 14, 2020
Summary:
Every time I run `yarn upgrade` on a personal project it spits out a warning:
```
warning react-relay > fbjs > [email protected]: core-js@<3 is no longer maintained and not
recommended for usage due to the number of issues. Please, upgrade your dependencies
to the actual version of core-js@3.
```

[New versions of fbjs and fbjs-scripts](facebook/fbjs@1f14496) were released back in July that included a PR to [upgrade `core-js` to version 3](facebook/fbjs#370).

This bumps the `fbjs` and `fbjs-scripts` versions to 2.0.0 for relay and all of its packages, which eliminates `core-js@<3` completely!

Pull Request resolved: #3184

Reviewed By: kassens

Differential Revision: D23667268

Pulled By: bigfootjon

fbshipit-source-id: f7ee028d6555b43af63b2e913e5db5ba78394a06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants