Skip to content
This repository was archived by the owner on Jul 13, 2020. It is now read-only.

Replace arrow function #1

Merged
merged 1 commit into from
Aug 19, 2016
Merged

Replace arrow function #1

merged 1 commit into from
Aug 19, 2016

Conversation

probins
Copy link
Contributor

@probins probins commented Aug 16, 2016

This solitary piece of ES6 prevents the file being compressed by standard UglifyJS, so I'd suggest removing it.

The other change was done auto by my editor :-)

Are you planning on releasing the dist/compressed version of this on jspm registry so people can use that rather than install their own copy?

@guybedford guybedford merged commit c28e12e into ModuleLoader:master Aug 19, 2016
@guybedford
Copy link
Member

Thanks. The distribution version is on npm to use.

@probins probins deleted the patch-1 branch August 19, 2016 08:59
@probins
Copy link
Contributor Author

probins commented Aug 19, 2016

what's the url? https://npm.jspm.io/[email protected]/system-register-loader.js isn't found

@guybedford
Copy link
Member

It's dist/system-register-loader.js... thanks, I had the wrong main set in the package.json.

@probins
Copy link
Contributor Author

probins commented Aug 19, 2016

uncompressed though ... And it won't work in the browser as it uses require("process") (looks like jspm.io is adding this)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants