Skip to content
This repository was archived by the owner on Sep 17, 2024. It is now read-only.

Update README.md to include Next.js 13+ bundling#200

Merged
JonasBa merged 1 commit intogetsentry:mainfrom
nalejandroveron:main
Oct 16, 2023
Merged

Update README.md to include Next.js 13+ bundling#200
JonasBa merged 1 commit intogetsentry:mainfrom
nalejandroveron:main

Conversation

@nalejandroveron
Copy link
Copy Markdown
Contributor

When trying to run the profiler in Nextjs 13+ I've found that I can't edit directly the "externals" because it' being passed as a function, but this setting exists to declare external packages, which looks to be working on my end (local dev, local build and deploying).

https://nextjs.org/docs/app/api-reference/next-config-js/serverComponentsExternalPackages


Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes.

@nalejandroveron nalejandroveron changed the title Update README.md Update README.md to include Next.js 13+ bundling Sep 27, 2023
@AbhiPrasad AbhiPrasad requested a review from JonasBa October 6, 2023 14:53
@JonasBa
Copy link
Copy Markdown
Member

JonasBa commented Oct 6, 2023

Thank you @Negan1911. I'm sorry, I must have somehow missed the notification (appreciate the tag @AbhiPrasad)

@JonasBa JonasBa merged commit 7a32fc1 into getsentry:main Oct 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants