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

feat: add browser ESM build #2590

Closed
wants to merge 1 commit into from
Closed

feat: add browser ESM build #2590

wants to merge 1 commit into from

Conversation

zigomir
Copy link
Contributor

@zigomir zigomir commented Jan 22, 2019

POC

fixes #2589

@zigomir
Copy link
Contributor Author

zigomir commented Jan 22, 2019

Still need to figure out how to uglify / minify. Same goes for vuejs/vuex#1401 I believe.

@posva
Copy link
Member

posva commented Jan 22, 2019

Thanks for initiating this!

@zigomir
Copy link
Contributor Author

zigomir commented Jan 22, 2019

@posva np. I'm sure someone can point out right direction here regarding uglify and traspilation. Are you maybe more familiar with build setup and what needs changing?

@Anoesj
Copy link

Anoesj commented Apr 10, 2019

Is anyone still working on this? This would be a very useful addition to vue-router!

Still need to figure out how to uglify / minify. Same goes for vuejs/vuex#1401 I believe.

Have you tried terser?

@zigomir
Copy link
Contributor Author

zigomir commented Apr 10, 2019

@Anoesj thanks! I didin't even know about it and I see vue is already just using terser instead of uglify-js.

Let me revive this PR with a working code.

@zigomir
Copy link
Contributor Author

zigomir commented Apr 11, 2019

Moved to #2705 – I deleted fork too soon.

@zigomir zigomir closed this Apr 11, 2019
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.

Browser ES module build
3 participants