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

Open
wants to merge 2 commits into
base: danf/crons-write-detector
Choose a base branch
from

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.

@wedamija wedamija force-pushed the danf/crons-write-detector branch from a37d0c5 to aada4a3 Compare August 9, 2025 00:01
Add a simple function to get the detector associated with a cron monitor, if one exists.
@wedamija wedamija force-pushed the danf/crons-get-detector branch from 1c4782e to 4975f30 Compare August 9, 2025 00:01
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                      @@
##           danf/crons-write-detector   #97547      +/-   ##
=============================================================
- Coverage                      80.62%   80.62%   -0.01%     
=============================================================
  Files                           8560     8560              
  Lines                         376933   376938       +5     
  Branches                       24538    24538              
=============================================================
+ Hits                          303916   303917       +1     
- Misses                         72647    72651       +4     
  Partials                         370      370              

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.

1 participant