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

Adds the ability to generate a project using ES2015 template #174

Closed
wants to merge 3 commits into from

Conversation

bluzi
Copy link

@bluzi bluzi commented Oct 11, 2017

Hi guys,

I added ES2015 support to the generator exactly as discussed in this issue
The generator detects whether it is running on a Node version that supports ES2015, and if it does, it will generate a project using the ES2015 templates. Otherwise, it will generate a project using the good old ES5 templates.

I also added two additional flags: --es5 and --es2015 - using one of them will force the generator to use one of these templates, regardless of the Node version.

The ES2015 support detection is provided using feature-detect-es6. I tested it a lot and it works flawlessly.

I tested it using the following Node versions, and it works as expected:

  • v0.10.48
  • v4.8.4
  • v6.11.4
  • v8.6.0

Please let me know if you want any changes, or if you have any feature suggestions or anything like that.

@bluzi
Copy link
Author

bluzi commented Oct 15, 2017

Looks like AppVeyor fails because of the changes I've made in README.md. That's weird. :/

@dougwilson
Copy link
Contributor

This is awesome, thank you! I would really love to get it included in the 4.16 release I was working on before you made your PR, but I haven't released yet. I changed the PR to base on the 4.16 branch, but it looks like there are conflicts it would be great if you could help resolve.

Also, can you add tests for the new command line switches?

@bluzi
Copy link
Author

bluzi commented Oct 21, 2017

Sure

@bluzi
Copy link
Author

bluzi commented Oct 21, 2017

I'm closing this PR, since I created new one with the changes you requested.
Please refer to the new PR

@bluzi bluzi closed this Oct 21, 2017
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.

None yet

2 participants