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

Switch babel preset to metro-react-native-babel-preset #20653

Closed
wants to merge 1 commit into from

Conversation

LinusU
Copy link
Contributor

@LinusU LinusU commented Aug 13, 2018

Fixes #20567, #20327

Test Plan:

Test to create a bare-bones project and run it on a device.

Release Notes:

Help reviewers and the release process by writing your own release notes. See below for an example.

[GENERAL] [BUGFIX] [Babel] - Switch templated babel preset from babel-preset-react-native to metro-react-native-babel-preset

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 13, 2018
Copy link
Contributor

@kelset kelset left a comment

Choose a reason for hiding this comment

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

LGTM

@kelset
Copy link
Contributor

kelset commented Aug 13, 2018

(not really sure why CI is failing though)

@janicduplessis
Copy link
Contributor

Thanks!

@facebook-github-bot shipit

@facebook-github-bot facebook-github-bot added the Import Started This pull request has been imported. This does not imply the PR has been approved. label Aug 13, 2018
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

janicduplessis is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@react-native-bot
Copy link
Collaborator

This pull request was closed by @LinusU in 35bd17f.

Once this commit is added to a release, you will see the corresponding version tag below the description at 35bd17f. If the commit has a single master tag, it is not yet part of a release.

@facebook facebook locked as resolved and limited conversation to collaborators Aug 13, 2018
@react-native-bot react-native-bot added the Merged This PR has been merged. label Aug 13, 2018
kelset pushed a commit that referenced this pull request Aug 13, 2018
Summary:
Fixes #20567
Pull Request resolved: #20653

Differential Revision: D9296819

Pulled By: hramos

fbshipit-source-id: 88953afb9778e3c8cde5576b068888fd882c2220
@LinusU LinusU deleted the babel-preset branch August 14, 2018 08:27
grabbou pushed a commit to react-native-community/cli that referenced this pull request Sep 26, 2018
Summary:
Fixes #20567
Pull Request resolved: facebook/react-native#20653

Differential Revision: D9296819

Pulled By: hramos

fbshipit-source-id: 88953afb9778e3c8cde5576b068888fd882c2220
@hramos hramos removed Import Started This pull request has been imported. This does not imply the PR has been approved. labels Feb 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants