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 ESM bundle filename and remove UMD bundle reference #14

Merged
merged 1 commit into from
Aug 30, 2019
Merged

Fix ESM bundle filename and remove UMD bundle reference #14

merged 1 commit into from
Aug 30, 2019

Conversation

sisp
Copy link
Contributor

@sisp sisp commented Aug 30, 2019

The ESM bundle filename in packages/lib/package.json does not match the actually built filename and there is a reference to a UMD bundle which currently is not built. Would you like to build a UMD bundle, too? This needs to be explicitly enabled since TSDX v0.7.0.

@netlify
Copy link

netlify bot commented Aug 30, 2019

Deploy preview for mobx-keystone ready!

Built with commit 472d89a

https://deploy-preview-14--mobx-keystone.netlify.com

@xaviergonz
Copy link
Owner

Thanks, I din't realize
I'm not sure if it is worth making an umd bundle, does anybody even use those anymore?

@xaviergonz xaviergonz merged commit 1e8e3db into xaviergonz:master Aug 30, 2019
@sisp
Copy link
Contributor Author

sisp commented Aug 30, 2019

I'm fine without UMD.

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.

2 participants