-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
package.json
/ dist/svgo.browser.js
are no longer exported
#1998
Comments
For example:
I can change the imports but this means that the changes in v3.3.0 are indeed breaking. :/ |
package.json
is no longer exportedpackage.json
/ dist/svgo.browser.js
are no longer exported
@SethFalco I just tried 4.0.0-rc here: jakearchibald/svgomg@b2cdb05 And it seems it doesn't work:
Did I miss something or something else is broken? Thanks! |
We don't export out SVGOMG only used it for the version I believe, so you can do, which is the reason I added it: const { VERSION } = require('svgo'); PS: A while back I opened a PR to update SVGO there too, but closed it later. In case it's helpful to you, these were the changes I proposed: |
Damn it, I misread the migration notes :) So, this works jakearchibald/svgomg@2c5835c Sorry for the noise! |
I don't follow the repos I have push rights so unless I'm CC'ed I won't see any issue :) I'm keeping the xmr-dev branch for now since I have other PRs pending, but I left them as is after reading the new plans. |
Don't be afraid to spam me if anything else comes up! And thanks for trying the RC and querying/reporting problems. Feedback on the release notes is also welcome if you think the issue was the wording. |
One thing I notice is the lack of extension in the browser export. Not a huge deal, but I prefer always adding an extension (plus xo errors by default for such cases). Your call though :) |
Sorry, could you clarify what this means? |
Using v3.3.1:
Adding this in the exports fixes the issue but this means that if people where requiring other things that the ones exported right now, they will be broken:
There must be a way to export everything again so that it's not a breaking change.
The text was updated successfully, but these errors were encountered: