migrate rollup v4 and plugins#3446
Conversation
|
@swissspidy Hello please run tests on PR |
|
remove manualChunks from tests because on v4.
|
|
Plz rerun |
|
rollup and rollup-plugins migrated to latests versions. |
|
@swissspidy PR is done.review plz |
swissspidy
left a comment
There was a problem hiding this comment.
Thank you! Looks reasonable from what I can tell.
|
this dependency |
Hi @husayt, a) Do you know perhaps if the test folder inside the plugin is needed / used somewhere? I would like to remove it from the npm releases, if they are useless in the final npm package - also improves the bundle size a lot. Is this enough for fixing the policy warnings? Are there a few more steps which can be done? I'm open for anything, as npm's security is very "stressed" right now (refering to the latest supply chain attacks) |
Fixes #3347