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

Change "main" to cjs instead #14

Closed
wants to merge 1 commit into from

Conversation

traeblain
Copy link
Contributor

I'm pretty sure only node needs the "main" term for the understanding of how the module is loaded. Everything else will use the js and min.js through direct pointers. As is, after doing an npm install temml, you cannot use the package in node because it wants to look at temml.js and exports an empty object.

I'm pretty sure only node needs the "main" term for the understanding of how the module is loaded.  Everything else will use the js and min.js through direct pointers.  As is, after doing an npm install temml, you cannot use the package in node because it wants to look at temml.js and exports an empty object.
@ronkok
Copy link
Owner

ronkok commented Jan 25, 2023

Well, my intent was to enable a web page to load Temml via jsDelivr. That's why main points to a file in iife format.

I certainly do want to accommodate people using node.js, but they can do that now by forking the Github repository.

I'm open to other suggestions so long as they do not interfere with the jsDelivr capability.

@traeblain
Copy link
Contributor Author

Found the answer so that it will work with both. Adding a new patch now.

@ronkok
Copy link
Owner

ronkok commented Jan 25, 2023

Not used in favor of #15.

@ronkok ronkok closed this Jan 25, 2023
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