Skip to content

chore(seer): Consolidate automation feature flags #91856

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
merged 2 commits into from
May 19, 2025
Merged

chore(seer): Consolidate automation feature flags #91856

merged 2 commits into from
May 19, 2025

Conversation

roaga
Copy link
Member

@roaga roaga commented May 17, 2025 •
edited
Loading

Switching everything from projects:trigger-issue-summary-on-alerts to organizations:trigger-autofix-on-issue-summary so that we only have one feature flag controlling all automation features. (names are meaningless now but we can handle that later)

Will remove the unused flag in a follow up

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 17, 2025
@codecov Codecov
Copy link

codecov bot commented May 17, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

⚠️ Parser warning

The parser emitted a warning. Please review your JUnit XML file:

Warning while parsing testcase attributes: Limit of string is 1000 chars, for name, we got 2083 at 1:156897 in /home/runner/work/sentry/sentry/.artifacts/pytest.junit.xml
Additional details and impacted files 
@@           Coverage Diff            @@
##           master   #91856    +/-   ##
========================================
  Coverage   87.62%   87.63%            
========================================
  Files       10353    10358     +5     
  Lines      586782   586672   -110     
  Branches    22586    22558    -28     
========================================
- Hits       514196   514127    -69     
+ Misses      72158    72119    -39     
+ Partials      428      426     -2

@roaga roaga marked this pull request as ready for review May 19, 2025 15:49
@roaga roaga requested review from a team as code owners May 19, 2025 15:49
@roaga roaga enabled auto-merge (squash) May 19, 2025 15:59
@roaga roaga merged commit 91db659 into master May 19, 2025
60 checks passed
@roaga roaga deleted the seer/consolidate-automation-ffs branch May 19, 2025 16:08
andrewshie-sentry pushed a commit that referenced this pull request May 20, 2025
@roaga @andrewshie-sentry
chore(seer): Consolidate automation feature flags (#91856)
84d10dc 
Switching everything from `projects:trigger-issue-summary-on-alerts` to
`organizations:trigger-autofix-on-issue-summary` so that we only have
one feature flag controlling all automation features. (names are
meaningless now but we can handle that later)

Will remove the unused flag in a follow up
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jennmueng jennmueng jennmueng approved these changes

Assignees
No one assigned
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@roaga @jennmueng