Skip to content

Conversation

@johanneslumpe
Copy link
Contributor

This is an edited re-submission of #1458 because I'm stupid.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Jun 3, 2015
Copy link
Contributor

Choose a reason for hiding this comment

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

This is only available on very recent node (0.12) and io.js versions. What's the supported version for the packager?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh that's odd. If we have to support older version, we might add

path.resolve(transformerPath) === path.normalize(transformerPath)

Copy link
Contributor

Choose a reason for hiding this comment

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

We want to support v0.10 (after all jest doesn't run on v0.12 yet). We have a package installed for this absolute-path

@amasad
Copy link
Contributor

amasad commented Jun 3, 2015

Just fix the absolute-path thing and this should be good to go.

@johanneslumpe
Copy link
Contributor Author

@amasad done!

@amasad
Copy link
Contributor

amasad commented Jun 4, 2015

Great! Will do pull it. It'd be nice to add some docs to that as well :) 💃

@amasad amasad self-assigned this Jun 4, 2015
@johanneslumpe
Copy link
Contributor Author

@amasad are those docs fine?

Copy link
Contributor

Choose a reason for hiding this comment

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

We don't really have to wait on all interpreters. We can just end the sentence at by allowing you to use new JavaScript language features.

@amasad
Copy link
Contributor

amasad commented Jun 15, 2015

Sorry for the late reply. Just a few nits on the docs

@johanneslumpe
Copy link
Contributor Author

@amasad updated the docs. I actually didn't write those texts, but they're changed the way you wanted now :)

@a2 a2 closed this in 1461e4a Jul 2, 2015
cpojer pushed a commit to facebook/metro that referenced this pull request Jan 26, 2017
Summary:
This is an edited re-submission of #1458 because I'm stupid.
Closes facebook/react-native#1497
Github Author: Johannes Lumpe <[email protected]>

Test Plan: Imported from GitHub, without a `Test Plan:` line.
ayushjainrksh pushed a commit to MLH-Fellowship/react-native that referenced this pull request Jul 2, 2020
simplify ActivityIndicator snack example
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.

5 participants