-
Notifications
You must be signed in to change notification settings - Fork 158
Move cron tick inside app server #104
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
Conversation
This change integrates the cron tick logic into the app server, removing external dependencies and enabling more frequent task processing. Co-authored-by: pierre <pierre@bazoge.com>
|
Cursor Agent can help with this pull request. Just |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #104 +/- ##
==========================================
+ Coverage 81.36% 81.44% +0.07%
==========================================
Files 151 152 +1
Lines 28276 28371 +95
==========================================
+ Hits 23007 23106 +99
+ Misses 4175 4170 -5
- Partials 1094 1095 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
Removes the need for external cron jobs by integrating a task scheduler directly into the application. This improves reliability and simplifies setup. Co-authored-by: pierre <pierre@bazoge.com>
Co-authored-by: pierre <pierre@bazoge.com>
This change decouples the TaskScheduler from the specific TaskService implementation by introducing a TaskExecutor interface. This allows for greater flexibility and testability. Unit tests have been added to cover the new interface and scheduler behavior. Co-authored-by: pierre <pierre@bazoge.com>
Replaces external cron job with an internal background scheduler for improved performance and simplified deployment. Co-authored-by: pierre <pierre@bazoge.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on November 30
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
This pull request contains changes generated by a Cursor Cloud Agent