Skip to content

ref(nextjs): Split withSentryConfig #18777

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.

Sign up for GitHub

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

Jump to bottom
Merged
chargome merged 3 commits into from
Jan 12, 2026
Merged

ref(nextjs): Split withSentryConfig #18777

chargome merged 3 commits into from
Jan 12, 2026

Conversation

@chargome
Copy link
Member

@chargome chargome commented Jan 12, 2026 •
edited by github-actions bot
Loading

The function became bloated and unreadable. This PR just splits the function and refactors parts into utils.

Closes #18778 (added automatically)
@chargome chargome self-assigned this Jan 12, 2026
@github-actions github-actions bot mentioned this pull request Jan 12, 2026
Closed
@chargome chargome requested a review from logaretm January 12, 2026 12:27
logaretm
logaretm approved these changes Jan 12, 2026
View reviewed changes
Copy link
Member

@logaretm logaretm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better!
@chargome chargome merged commit 0b0b904 into develop Jan 12, 2026
69 checks passed
@chargome chargome deleted the cg-next-refactor-withsentryconfig branch January 12, 2026 13:34
@github-actions github-actions bot mentioned this pull request Jan 13, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@chargome @logaretm