[CI] Run scheduled main workflow daily, skip after recent commits - #2082
Conversation
Run On Schedule every day at 02:00 UTC instead of Monday/Wednesday/Friday. Skip the CI jobs when a commit already landed on main in the last 24 hours, since On Push already covered that change. Manual workflow_dispatch still always runs.
|
Thanks @tdavidcl for opening this PR! You can do multiple things directly here: Once the workflow completes a message will appear displaying informations related to the run. Also the PR gets automatically reviewed by gemini, you can: |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan includes up to 1 review per rolling hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe scheduled workflow now runs daily at 02:00 UTC. A ChangesScheduled CI gating
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: 🔵 Low · up to The workflow may occasionally rerun CI unnecessarily or skip scheduled CI longer than intended because it uses commit timestamp rather than landing time for the 24-hour check. The change is mergeable with explicit owner awareness or follow-up to confirm that this timestamp is an acceptable proxy. Sequence Diagram(s)sequenceDiagram
participant GitHubActionsScheduler
participant should_run
participant CodecovSecretCheck
participant MainWorkflow
GitHubActionsScheduler->>should_run: Start daily scheduled workflow
should_run->>should_run: Check latest commit age
should_run-->>CodecovSecretCheck: Set run_ci
should_run-->>MainWorkflow: Set run_ci
CodecovSecretCheck->>CodecovSecretCheck: Run when run_ci is true
MainWorkflow->>MainWorkflow: Invoke when run_ci is true
Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Workflow reportworkflow report corresponding to commit f396426 Light CI is enabled (the default for pull requests). This will only run the basic tests and not the full tests. Pre-commit check reportPre-commit check: ✅ Test pipeline can run. Clang-tidy diff reportNo relevant changes found. You should now go back to your normal life and enjoy a hopefully sunny day while waiting for the review. Doxygen diff with
|
|
Queued — the merge queue status continues in this comment ↓. |
Merge Queue Status
This pull request spent 2 hours 3 minutes 46 seconds in the queue, including 1 hour 51 minutes 6 seconds running CI. Required conditions to merge
|
Run On Schedule every day at 02:00 UTC instead of Monday/Wednesday/Friday. Skip the CI jobs when a commit already landed on main in the last 24 hours, since On Push already covered that change. Manual workflow_dispatch still always runs.