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

ERR_PNPM_PEER_DEP_ISSUES  Unmet peer dependencies #7176

Closed
1 task
dougouverson opened this issue May 23, 2023 · 1 comment · Fixed by #7191
Closed
1 task

ERR_PNPM_PEER_DEP_ISSUES  Unmet peer dependencies #7176

dougouverson opened this issue May 23, 2023 · 1 comment · Fixed by #7191
Assignees
Labels
- P2: nice to have Not breaking anything but nice to have (priority) pkg: mdx Issues pertaining to `@astrojs/mdx` integration

Comments

@dougouverson
Copy link

What version of astro are you using?

2.5.3

Are you using an SSR adapter? If so, which one?

Netlify

What package manager are you using?

pnpm

What operating system are you using?

Mac

What browser are you using?

Firefox

Describe the Bug

When attempting to deploy to Netlify:

10:54:12 AM: Packages: +48
10:54:12 AM: ++++++++++++++++++++++++++++++++++++++++++++++++
10:54:12 AM: Progress: resolved 614, reused 0, downloaded 592, added 48, done
10:54:12 AM: dependencies:
10:54:12 AM: + @astrojs/mdx 0.19.3
10:54:12 AM:  ERR_PNPM_PEER_DEP_ISSUES  Unmet peer dependencies
10:54:12 AM: .
10:54:12 AM: └─┬ @astrojs/mdx 0.19.3
10:54:12 AM:   └─┬ @mdx-js/rollup 2.3.0
10:54:12 AM: Failed during stage 'Install dependencies': dependency_installation script returned non-zero exit code: 1
10:54:12 AM:     ├── ✕ missing peer rollup@>=2
10:54:12 AM:     └─┬ @rollup/pluginutils 5.0.2
10:54:12 AM:       └── ✕ missing peer rollup@^1.20.0||^2.0.0||^3.0.0
10:54:12 AM: Peer dependencies that should be installed:
10:54:12 AM:   rollup@">=2.0.0 <3.0.0 || >=3.0.0 <4.0.0"
10:54:12 AM: hint: If you want peer dependencies to be automatically installed, add "auto-install-peers=true" to an .npmrc file at the root of your project.
10:54:12 AM: hint: If you don't want pnpm to fail on peer dependency issues, add "strict-peer-dependencies=false" to an .npmrc file at the root of your project.
10:54:12 AM: Error during pnpm install
10:54:12 AM: Build was terminated: dependency_installation script returned non-zero exit code: 1
10:54:12 AM: Failing build: Failed to install dependencies

Link to Minimal Reproducible Example

https://stackblitz.com/

Participation

  • I am willing to submit a pull request for this issue.
@bluwy
Copy link
Member

bluwy commented May 24, 2023

You can workaround with this for now:

10:54:12 AM: hint: If you want peer dependencies to be automatically installed, add "auto-install-peers=true" to an .npmrc file at the root of your project.
10:54:12 AM: hint: If you don't want pnpm to fail on peer dependency issues, add "strict-peer-dependencies=false" to an .npmrc file at the root of your project.

I noticed we don't actually have to use the @mdx-js/rollup dep, so refactoring that away should fix it.

@bluwy bluwy added - P2: nice to have Not breaking anything but nice to have (priority) pkg: mdx Issues pertaining to `@astrojs/mdx` integration labels May 24, 2023
@bluwy bluwy self-assigned this May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- P2: nice to have Not breaking anything but nice to have (priority) pkg: mdx Issues pertaining to `@astrojs/mdx` integration
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants