Skip to content

Comments

fix: add support for Rollup version 3#125

Closed
alan-agius4 wants to merge 1 commit intomaxdavidson:masterfrom
alan-agius4:patch-1
Closed

fix: add support for Rollup version 3#125
alan-agius4 wants to merge 1 commit intomaxdavidson:masterfrom
alan-agius4:patch-1

Conversation

@alan-agius4
Copy link

This fixes incorrect peer deps

  +-- rollup-plugin-sourcemaps@0.6.3
  | +-- @rollup/pluginutils@3.1.0
  | | `-- rollup@3.5.1 deduped invalid: "^1.20.0||^2.0.0" from node_modules/rollup-plugin-sourcemaps/node_modules/@rollup/pluginutils
  | `-- rollup@3.5.1 deduped

This fixes incorrect peer deps

```
  +-- rollup-plugin-sourcemaps@0.6.3
  | +-- @rollup/pluginutils@3.1.0
  | | `-- rollup@3.5.1 deduped invalid: "^1.20.0||^2.0.0" from node_modules/rollup-plugin-sourcemaps/node_modules/@rollup/pluginutils
  | `-- rollup@3.5.1 deduped
```
@justingrant
Copy link

We just ran across this in an update to our repo @js-temporal/polyfill. Thanks @alan-agius4 for saving us the hassle of filing the same PR!

If anyone else has the same issue, before this PR is merged here's how we worked around it in our package.json:

  "overrides": {
    "@rollup/pluginutils": "^5.0.2"
  },

@segevfiner
Copy link

segevfiner commented Dec 25, 2022

Fixes #127

@janbiasi
Copy link

janbiasi commented Mar 7, 2023

@maxdavidson what blocks this PR from being merged?

@NetWin
Copy link

NetWin commented Nov 30, 2023

@maxdavidson an ETA? 😄

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants