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

build(configuration): module property within package.json #8

Merged
merged 4 commits into from
Feb 21, 2017

Conversation

nissoh
Copy link
Contributor

@nissoh nissoh commented Feb 5, 2017

I'm trying to create a bundled file so it can be served to the browser.
my final bundle ends up with es2015 code which cannot run in older versions of chrome, ff, IE.
is it being forced because of the module property?

My tsconfig.json module and target fields configured to commonjs (module code generation)
and es5. I removed the module field locally within my node_modules folder and the end
footprint is es5 as i configured through my own tsconfig.json.

I'm trying to create a bundled file so it can be served to the browser.

My `tsconfig.json` `module` and `target` fields configured to `commonjs` (module code generation)
and `es5`. I removed the `module` field locally within my `node_modules` folder and the end
footprint is `es5` as i configured through my own `tsconfig.json`.
@nissoh
Copy link
Contributor Author

nissoh commented Feb 17, 2017

@TylorS ping (:

@TylorS
Copy link
Member

TylorS commented Feb 17, 2017

Thanks for pinging me @nissoh, this had gotten away from me.

I've realized in some of my newer projects, but hadn't gotten back to making adjustments here. The "proper" fix, is to make an adjustment to the tsconfig.json's target field. It needs to be changed to es5 instead of es2015, leaving just the module to es2015. So the code generated is transpiled but imports/exports are kept.

Nissan Hanina added 3 commits February 21, 2017 09:35
@TylorS TylorS merged commit d94dabb into mostjs-community:master Feb 21, 2017
@TylorS
Copy link
Member

TylorS commented Feb 21, 2017

Fixed with v3.3.0

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

Successfully merging this pull request may close these issues.

None yet

2 participants