Skip to content

build(eslint): add curly rule to prettier config section#116158

Merged
ryan953 merged 2 commits into
masterfrom
junior/curly-eslint-rule
May 26, 2026
Merged

build(eslint): add curly rule to prettier config section#116158
ryan953 merged 2 commits into
masterfrom
junior/curly-eslint-rule

Conversation

@sentry-junior
Copy link
Copy Markdown
Contributor

@sentry-junior sentry-junior Bot commented May 26, 2026

Adds curly: 'error' to the prettier config block in eslint.config.ts.

Ran pnpm fix:eslint — no violations found across the codebase, so this is a config-only change.

Co-authored-by: Ryan Albrecht ryan.albrecht@sentry.io


View Session in Sentry

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 26, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.58%

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

Co-authored-by: Ryan Albrecht <ryan@ryanalbrecht.ca>
@ryan953 ryan953 added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 26, 2026
@ryan953 ryan953 marked this pull request as ready for review May 26, 2026 13:43
@ryan953 ryan953 requested review from a team as code owners May 26, 2026 13:43
@evanpurkhiser
Copy link
Copy Markdown
Member

merge it

@ryan953 ryan953 merged commit afc2a6a into master May 26, 2026
87 of 88 checks passed
@ryan953 ryan953 deleted the junior/curly-eslint-rule branch May 26, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants