Skip to content

fix(ci): let precommit format json files#111874

Merged
TkDodo merged 1 commit intomasterfrom
tkdodo/fix/precommit-formatting-json-files
Mar 31, 2026
Merged

fix(ci): let precommit format json files#111874
TkDodo merged 1 commit intomasterfrom
tkdodo/fix/precommit-formatting-json-files

Conversation

@TkDodo
Copy link
Copy Markdown
Collaborator

@TkDodo TkDodo commented Mar 31, 2026

No description provided.

@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Mar 31, 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.

Copy link
Copy Markdown
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

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

lgtm

@TkDodo TkDodo marked this pull request as ready for review March 31, 2026 10:33
@TkDodo TkDodo requested review from a team as code owners March 31, 2026 10:33
@TkDodo TkDodo merged commit 56226c4 into master Mar 31, 2026
87 checks passed
@TkDodo TkDodo deleted the tkdodo/fix/precommit-formatting-json-files branch March 31, 2026 10:34
@github-actions github-actions bot locked and limited conversation to collaborators Apr 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants