Skip to content

chore(ACI): Replace multiple flag usage with single flag#113461

Merged
ceorourke merged 2 commits intomasterfrom
ceorourke/swap-metric-flags-one-flag
Apr 20, 2026
Merged

chore(ACI): Replace multiple flag usage with single flag#113461
ceorourke merged 2 commits intomasterfrom
ceorourke/swap-metric-flags-one-flag

Conversation

@ceorourke
Copy link
Copy Markdown
Member

Condense 6 flags into 1 - all of these are individually GA'd and the replacement flag is GA'd too (once https://github.com/getsentry/sentry-options-automator/pull/7370 is merged).

@ceorourke ceorourke requested a review from a team as a code owner April 20, 2026 18:35
@ceorourke ceorourke requested a review from a team April 20, 2026 18:35
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 20, 2026
@ceorourke ceorourke requested a review from kcons April 20, 2026 18:36
Comment thread src/sentry/incidents/endpoints/organization_alert_rule_details.py Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6d2070b. Configure here.

Comment thread src/sentry/incidents/endpoints/organization_alert_rule_details.py Outdated
Comment thread src/sentry/incidents/endpoints/organization_incident_details.py Outdated
@ceorourke ceorourke merged commit 7671894 into master Apr 20, 2026
77 checks passed
@ceorourke ceorourke deleted the ceorourke/swap-metric-flags-one-flag branch April 20, 2026 19:14
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants