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

No longer resolves against 'browser' field #12

Closed
tmcw opened this issue May 11, 2018 · 0 comments
Closed

No longer resolves against 'browser' field #12

tmcw opened this issue May 11, 2018 · 0 comments

Comments

@tmcw
Copy link
Member

tmcw commented May 11, 2018

Noticed that including simple-statistics is broken with d3-require 1.0.1 - this appears to be a result of simple-statistics relying on the browser field. I'll shortly add an unpkg field to simple-statistics to nail this down, but nothing that this is technically a regression from unpkg's (admittedly undocumented) behavior: it redirects

https://unpkg.com/simple-statistics -> https://unpkg.com/simple-statistics@6.0.0/dist/simple-statistics.min.js

whereas d3-require currently doesn't use unpkg's auto-resolution of a main file, and instead loads

https://unpkg.com/simple-statistics@6.0.0/dist/simple-statistics.js

From the main field in package.json, which is not browser-compatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant