Skip to content
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.

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

fix(aci milestone 3): separate flag for processing metric issue workflows #89443

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mifu67
Copy link
Contributor

@mifu67 mifu67 commented Apr 11, 2025

Gating processing of both metric issues and errors with one flag was causing problems if an org was flagged into metric detectors but not error detectors (or vice versa). Add a separate flag for metric issues.

@mifu67 mifu67 requested review from a team as code owners April 11, 2025 22:27
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 11, 2025
Copy link

codecov bot commented Apr 11, 2025

Codecov Report

Attention: Patch coverage is 71.42857% with 6 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/tasks/post_process.py 68.42% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #89443      +/-   ##
==========================================
- Coverage   87.70%   86.50%   -1.20%     
==========================================
  Files       10144    10147       +3     
  Lines      573277   573364      +87     
  Branches    22525    22525              
==========================================
- Hits       502811   496015    -6796     
- Misses      70032    76915    +6883     
  Partials      434      434              

Copy link
Contributor

@saponifi3d saponifi3d left a comment

Choose a reason for hiding this comment

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

lgtm - thanks for the refactor! 🎉

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