-
Notifications
You must be signed in to change notification settings - Fork 71
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
esm support #93
Comments
FWIW - I forked and created https://www.npmjs.com/package/@bengreenier/sdp-transform (gh link) for my needs, I'm no ESM expert but I think it's sufficient for most ESM use cases. |
why didn't you just send a PR and stick to plain vanilla javascript and some jsDoc? i think typescript syntax is unnecessary, more complicated and require compiling, which is unnecessary, not the original code, so it's then also harder to debug. |
I wanted my fork to be typescript 🤷♂️ feel free to ignore it and/or do those things yourself. |
@bengreenier does your fork have a release I could just download and import without NPM? |
Like a standalone GitHub release? Unfortunately it currently does not. Though I suppose you could grab the tarball from npm, same as you could for this original package. Maybe something like https://stackoverflow.com/questions/15035786/download-source-from-npm-without-installing-it would meet your needs? |
The ESM support is required for this package to be compatible with the new Symfony's AssetMapper. I just tried to install a package depending on this one, and the installation failed because of the lack of the ESM support. This will be happening more and more often, as modules are getting more popular in the place of bundles. |
Would very much like to just simply use (async) import from a CDN without bundling the code
The text was updated successfully, but these errors were encountered: