Skip to content

fix(alerts): Update logger string to be accurate #77603

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

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

ceorourke
Copy link
Member

Oops this is the store data endpoint, not detect anomalies

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 16, 2024
@ceorourke ceorourke requested a review from a team September 16, 2024 22:50
Copy link

codecov bot commented Sep 16, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/seer/anomaly_detection/store_data.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master   #77603   +/-   ##
=======================================
  Coverage   78.11%   78.11%           
=======================================
  Files        6959     6959           
  Lines      308865   308865           
  Branches    50544    50544           
=======================================
+ Hits       241277   241282    +5     
+ Misses      61187    61183    -4     
+ Partials     6401     6400    -1     

@ceorourke ceorourke merged commit 3e35394 into master Sep 17, 2024
50 of 51 checks passed
@ceorourke ceorourke deleted the ceorourke/update-logger-anomaly-detection branch September 17, 2024 16:35
@github-actions github-actions bot locked and limited conversation to collaborators Oct 3, 2024
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants