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

feat(gatsby): Support Gatsby v5 #6635

Merged
merged 1 commit into from
Jan 4, 2023
Merged

feat(gatsby): Support Gatsby v5 #6635

merged 1 commit into from
Jan 4, 2023

Conversation

rjoonas
Copy link
Contributor

@rjoonas rjoonas commented Jan 2, 2023

At least the Gatsby v5 release notes mention no changes to the plugin interfaces used here (gatsby-browser#onClientEntry and gatsby-node#onCreateWebpackConfig).

The only instruction for plugin maintainers in the v4 to v5 migration guide is that ”In most cases, you won’t have to do anything to be v5 compatible.”

Resolves #6261.


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 (yarn lint) & (yarn test).

@Lms24 Lms24 requested review from a team, mydea, lforst and Lms24 and removed request for a team January 3, 2023 09:02
@Lms24
Copy link
Member

Lms24 commented Jan 3, 2023

Hi @rjoonas, thanks for opening this PR! Did you have any chance to test the SDK in v5? I'm curious if you noticed any obvious problems or odd behaviour.

@rjoonas
Copy link
Contributor Author

rjoonas commented Jan 3, 2023

Hi @rjoonas, thanks for opening this PR! Did you have any chance to test the SDK in v5? I'm curious if you noticed any obvious problems or odd behaviour.

Tried creating a fresh Gatsby 5 project using npm init gatsby, installing the plugin and manually throwing some test exceptions to report in dev mode. No obvious problems. Site also builds without errors.

@lforst
Copy link
Member

lforst commented Jan 4, 2023

Thank you! :)

@lforst lforst merged commit 72d77d8 into getsentry:master Jan 4, 2023
@lforst
Copy link
Member

lforst commented Jan 10, 2023

We just released this change with version 7.30.0 of the SDK.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sentry/gatsby doesn't support Gatsby 5 as a peer dependency
3 participants