chore(nextjs): Move withSentry
into withSentryAPI
module
#5801
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This moves
withSentry.ts
(and its corresponding test file) to live alongside our otherwithSentryX
wrappers. More specifically, it moveswithSentry
's types into a newwrappers/types
file, and moves its functions intowithSentryAPI
, sincewithSentryAPI
is becoming the primary wrapper andwithSentry
is becoming just a helper function.Note: This is one step in a slightly larger reorganization I want to do in the nextjs SDK, now that it's matured a bit and we have a better idea of what files it needs to contain. For both ease of review and my own time management, I'm going to split the changes up over several PRs and do each as they become relevant and/or start to bug me enough that I fix them.