Add missing peer dependencies to Gatsby package#4550
Conversation
Fixes #4549 by copying missing peer dependencies from `packages/nextjs/package.json`.
|
Hey thanks for making a PR! We are current discussing peer deps in webpack plugin, so we are holding off on merging this. cc @kamilogorek |
|
No problem, and it's easily fixable locally with Yarn 3 |
|
We'll be addressing the webpack dep here: getsentry/sentry-webpack-plugin#350, so closing this PR. Thanks for the help! |
|
Thanks @AbhiPrasad. What about the React dependency? |
|
Ah that’s my bad, we can re-open to take care of react! |
|
No problem - i'll file a new one-line PR now! Edit: #4576 |
|
@AbhiPrasad, it looks like the making-Webpack-optional got reverted (IMO correctly). Again, my repo is fine thanks to |
|
Yup, we should get to that, we'll update when we upgrade the webpack plugin dep across the board. |
Fixes #4549 by copying missing peer dependencies from
packages/nextjs/package.json.Before submitting a pull request, please take a look at our
Contributing guidelines and verify:
yarn lint) & (yarn test).