Skip to content

fix(nextjs): webpack as optional peer-dependency#4634

Merged
AbhiPrasad merged 1 commit intogetsentry:masterfrom
belgattitude:sentry-nextjs-webpack-optional
Feb 25, 2022
Merged

fix(nextjs): webpack as optional peer-dependency#4634
AbhiPrasad merged 1 commit intogetsentry:masterfrom
belgattitude:sentry-nextjs-webpack-optional

Conversation

@belgattitude
Copy link
Copy Markdown
Contributor

@belgattitude belgattitude commented Feb 24, 2022

Should close #4632

  • fix a warning about a missing peer-dep on webpack.
  • should fix install with yarn pnp / pnpm modules linkers

Notes

@AbhiPrasad
Copy link
Copy Markdown
Contributor

Hey thanks for opening a PR. We're currently discussing this in #4635, feedback welcome!

@AbhiPrasad AbhiPrasad changed the title chore(sentry/nextjs): webpack as optional peer-dependency fix(nextjs): webpack as optional peer-dependency Feb 25, 2022
@AbhiPrasad
Copy link
Copy Markdown
Contributor

Let's go with this for now, and re-evaluate as needed. Thanks for the help!

@AbhiPrasad AbhiPrasad merged commit ad76379 into getsentry:master Feb 25, 2022
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.

sentry/nextjs warning on install about missing webpack

2 participants