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

Document new es6module flag in react-tools README #3618

Merged
merged 1 commit into from
Apr 8, 2015

Conversation

TimeBomb
Copy link
Contributor

@TimeBomb TimeBomb commented Apr 8, 2015

This pull request documents the new JSXTransformer es6module and nonStrictEs6module flags added a couple months ago by @jeffmo.

This code was documented per the code comments given in the original PR, #3586.

It would have saved me some time if this were documented, as I was having issues getting code using import/export to compile. Better late than never.

Note that I have completed the Facebook CLA.

Cheers!

@@ -41,6 +43,8 @@ option | values | default
`harmony` | `true`: enable ES6 features | `false`
`sourceFilename` | the output filename for the source map | `"source.js"`
`stripTypes` | `true`: strips out type annotations | `false`
`es6module` | `true`: parses the file as an ES6 module | 'false'
`nonStrictEs6module` | `true`: parses the file as an ES6 module, except disables implicit strict-mode (i.e. CommonJS modules et al are allowed) | 'false'
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Use backticks for false

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

zpao added a commit that referenced this pull request Apr 8, 2015
Document new es6module flag in react-tools README
@zpao zpao merged commit cf76365 into facebook:master Apr 8, 2015
@zpao
Copy link
Member

zpao commented Apr 8, 2015

Looks good, thanks! Sorry that you lost some time :(

@zpao zpao added this to the 0.13.2 milestone Apr 8, 2015
zpao added a commit to zpao/react that referenced this pull request Apr 13, 2015
Document new es6module flag in react-tools README
zpao added a commit to zpao/react that referenced this pull request Apr 17, 2015
Document new es6module flag in react-tools README
zpao added a commit to zpao/react that referenced this pull request Apr 18, 2015
Document new es6module flag in react-tools README
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