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

bundle Prism in the ESM version to prevent Vite issues #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mesqueeb
Copy link

@mesqueeb mesqueeb commented Jul 19, 2021

fixes: #40

PS: we can remove --bundle-node-modules again once the ESM version of Prism comes out.

PS: we can remove `--bundle-node-modules` again once the ESM version of Prism comes out.
@mesqueeb
Copy link
Author

fixes: #40

@mesqueeb
Copy link
Author

@egoist kindly review this small PR with great benefit : )

@Nisgrak
Copy link

Nisgrak commented Sep 27, 2021

After SO MUCH debugging because my own lib didn't work in Vite this is the problem.... pls @egoist review and accept this

@Nisgrak
Copy link

Nisgrak commented Sep 27, 2021

I create a fork with the dist of your fork, so anyone can install directly https://github.com/Nisgrak/vue-prism-component

npm i https://github.com/Nisgrak/vue-prism-component
yarn add https://github.com/Nisgrak/vue-prism-component

@mesqueeb
Copy link
Author

mesqueeb commented Sep 27, 2021

@Nisgrak i also wasted about 4+ hours on this issue, but I learned alittre bit while finding out what exactly was going wrong. : )

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.

vue-prism-component does not provide an export named 'default'
2 participants