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
Bump rollup from 2 to 3 #1949
Bump rollup from 2 to 3 #1949
Conversation
Bundle size report: Size Change: -29 B
ℹ️ View Details
|
Why the |
There was an issue with import asserts in the docs repo. Do you think the new rollup version might solve the problems we have seen there too? |
I think so, there's a need to make sure the import now uses the |
Right now we run the 'remove-assert.js' script in the docs as well on the style-spec |
I'd like to test these changes against the docs, but we have this situation where it has to be released from maplibre-gl-js before it can be built with the docs. |
Dependabot can't handle major updates. Rollup 3 requires us to use an extra "--bundleConfigAsCjs" flag in package.json, and it has built-in support for import assertions with @rollup/plugin-json, so the import assertion plugins are removed.