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

Handle the case where transformSync returns null #264

Closed

Conversation

jkimbo
Copy link
Contributor

@jkimbo jkimbo commented Sep 24, 2018

Summary

Fixes a bug when ignoring a file in the babel config: #255 (comment)

Test plan

Couldn't find any obvious tests that test this part of the code. Happy to add some tests if someone can point me in the right direction.

@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 Sep 24, 2018
@codecov-io
Copy link

Codecov Report

Merging #264 into master will decrease coverage by 0.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #264      +/-   ##
==========================================
- Coverage   85.68%   85.67%   -0.02%     
==========================================
  Files         144      144              
  Lines        4556     4558       +2     
  Branches      712      713       +1     
==========================================
+ Hits         3904     3905       +1     
- Misses        582      583       +1     
  Partials       70       70
Impacted Files Coverage Δ
packages/metro/src/reactNativeTransformer.js 90% <75%> (-1.67%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bba48f0...d2573e3. Read the comment docs.

Copy link
Contributor

@rafeca rafeca left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! LGTM 😃

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.

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

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants