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

Fix package.json exports #91

Merged
merged 1 commit into from
Jul 11, 2022
Merged

Fix package.json exports #91

merged 1 commit into from
Jul 11, 2022

Conversation

gesi
Copy link
Contributor

@gesi gesi commented Jul 8, 2022

I was running into some problems using this package with svelte kit/vite when building for production. The root of my problem seems to be that the ECMAScript module loader does not support directories/folders as modules. See https://nodejs.org/api/packages.html.

Also, this might be related to #89.

@enyo
Copy link

enyo commented Jul 10, 2022

@yiliansource it would be great if you could merge this, since it's preventing me from using your library. Thanks!

@yiliansource
Copy link
Owner

I was not aware that this would cause issues, thanks!

Do you know if this change would cause any compatibility issues as a patch?

@gesi
Copy link
Contributor Author

gesi commented Jul 10, 2022

Do you know if this change would cause any compatibility issues as a patch?

I can't imagine this change would break anything

@enyo
Copy link

enyo commented Jul 11, 2022

I also think that this is safe to merge.

@enyo
Copy link

enyo commented Jul 11, 2022

@yiliansource thank you! It works now.

@Santosl2
Copy link

Wooww! Thanks dude

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.

4 participants