kentcdodds / mdx-bundler

🦤 Give me MDX/TSX strings and I'll give you back a component you can render. Supports imports!
MIT License
1.75k stars 75 forks source link

Dependency issue #187

Open kwiat1990 opened 2 years ago

kwiat1990 commented 2 years ago

What you did: I wanted to update Remix to newest version 1.7.0.

What happened: I'm getting following error from npm:

Command failed: npm install --save-dev @remix-run/dev@1.7.0 @remix-run/eslint-config@1.7.0 eslint@8.23.0 --color=always
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR! 
npm ERR! While resolving: mdx-bundler@9.0.1
npm ERR! Found: esbuild@0.15.6
npm ERR! node_modules/esbuild
npm ERR!   peer esbuild@"*" from @esbuild-plugins/node-modules-polyfill@0.1.4
npm ERR!   node_modules/@esbuild-plugins/node-modules-polyfill
npm ERR!     @esbuild-plugins/node-modules-polyfill@"^0.1.4" from @remix-run/dev@1.7.0
npm ERR!     node_modules/@remix-run/dev
npm ERR!       dev @remix-run/dev@"1.7.0" from the root project
npm ERR!   peer esbuild@"*" from @esbuild-plugins/node-resolve@0.1.4
npm ERR!   node_modules/@esbuild-plugins/node-resolve
npm ERR!     @esbuild-plugins/node-resolve@"^0.1.4" from mdx-bundler@9.0.1
npm ERR!     node_modules/mdx-bundler
npm ERR!       mdx-bundler@"^9.0.1" from the root project
npm ERR!   3 more (@mdx-js/esbuild, @yarnpkg/esbuild-plugin-pnp, the root project)
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peer esbuild@"0.11.x || 0.12.x || 0.13.x || 0.14.x" from mdx-bundler@9.0.1
npm ERR! node_modules/mdx-bundler
npm ERR!   mdx-bundler@"^9.0.1" from the root project
npm ERR! 
npm ERR! Conflicting peer dependency: esbuild@0.14.54
npm ERR! node_modules/esbuild
npm ERR!   peer esbuild@"0.11.x || 0.12.x || 0.13.x || 0.14.x" from mdx-bundler@9.0.1
npm ERR!   node_modules/mdx-bundler
npm ERR!     mdx-bundler@"^9.0.1" from the root project
bitgoAaron commented 1 year ago

Can we get a new release to bump mdx-bundler's version requirements to include es-build@0.15.x ?

NateRadebaugh commented 1 year ago

Looks like the following PR may be all we need: #188

magoz commented 1 year ago

Same here