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

Maybe provide a "main", not just a "module" #6

Closed
zzolo opened this issue Dec 23, 2018 · 3 comments
Closed

Maybe provide a "main", not just a "module" #6

zzolo opened this issue Dec 23, 2018 · 3 comments

Comments

@zzolo
Copy link

zzolo commented Dec 23, 2018

esm doesn't seem to work with this module.

This looks like it may be a bug or I may just have something way off, but it might be useful to provide a main field, and not just a module field.

And it might be worth updating the documentation that the module is only provided in ES module syntax.

Thanks for the help and great library.

@zzolo zzolo changed the title Maybe proivde a "main", not just a "module" Maybe provide a "main", not just a "module" Dec 23, 2018
@mhkeller
Copy link
Owner

I'll add that and push a patch version to see if it fixes things for you. I wonder if the svelte-component setup is the right way to compile it. What is your environment you're using it in?

@mhkeller
Copy link
Owner

0.3.1 has the main field!

@zzolo
Copy link
Author

zzolo commented Dec 24, 2018

Awesome! This worked for me well.

@zzolo zzolo closed this as completed Dec 24, 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

2 participants