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

Open to an ES5 transpile build step? #16

Open
guyellis opened this issue Jun 17, 2016 · 5 comments
Open

Open to an ES5 transpile build step? #16

guyellis opened this issue Jun 17, 2016 · 5 comments

Comments

@guyellis
Copy link

If I (or someone) can work out a way to do it would you be open to accepting a PR that has a prepublish step that transpiles the ES6 code to ES5 so that ES5 restrained consumers can also use this module?

E.g. prepublish uses babel to generate output to /lib/es5/ before publish and that's the entry point for ES5 consumers. This would not impact the ES6 consumers and the current ES6 being used in the module. i.e. ES6 by default. ES5 needs to be explicitly required by the consumer by path.

Might also help with #1 which might need ES5 for browsers.

@guyellis guyellis changed the title Open to an ES5 transpile build step Open to an ES5 transpile build step? Jun 17, 2016
@sveisvei
Copy link
Collaborator

Shipping a es5 version should be fine, and is probably needed for a universal version. I am all for it atleast

@jcchavezs
Copy link
Contributor

Any input @DavideCarvalho @sonybinhle?

@DavideCarvalho
Copy link

I started a branch for this task. I think the best tech for this is rollup, since we can transpile for es5 and es6 at the same time. If someone needs to target only evergreen targets, they can use es6 build (and use stuff like type=module for scripts), otherwise, use es5.

@sonybinhle
Copy link
Collaborator

I am for rollup also.

@jcchavezs
Copy link
Contributor

jcchavezs commented Nov 26, 2018 via email

@jcchavezs jcchavezs mentioned this issue Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants