Skip to content

feat(crons): Add get_detector_for_monitor function #97547

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 6 commits into from
Aug 12, 2025

Conversation

wedamija
Copy link
Member

@wedamija wedamija commented Aug 8, 2025

Add a simple function to get the detector associated with a cron monitor, if one exists.

@wedamija wedamija requested a review from a team August 8, 2025 23:45
@wedamija wedamija requested a review from a team as a code owner August 8, 2025 23:45
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 8, 2025
cursor[bot]

This comment was marked as outdated.

Copy link

codecov bot commented Aug 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #97547      +/-   ##
==========================================
- Coverage   80.65%   80.65%   -0.01%     
==========================================
  Files        8571     8571              
  Lines      377690   377695       +5     
  Branches    24583    24583              
==========================================
+ Hits       304613   304617       +4     
- Misses      72711    72712       +1     
  Partials      366      366              

cursor[bot]

This comment was marked as outdated.

@wedamija wedamija force-pushed the danf/crons-get-detector branch from d1febb3 to c87fb66 Compare August 12, 2025 16:52
@wedamija wedamija merged commit 1d46292 into master Aug 12, 2025
63 of 64 checks passed
@wedamija wedamija deleted the danf/crons-get-detector branch August 12, 2025 17:21
andrewshie-sentry pushed a commit that referenced this pull request Aug 12, 2025
Add a simple function to get the detector associated with a cron
monitor, if one exists.
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