Repository navigation
Run scheduled shell drift checks twice a month - #8
Merged
Merged
Conversation
Compare the shared site shell on the 1st and 15th at 07:23 UTC while retaining manual and source/PR checks.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Docstring Coverage |  |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
| Description Check | |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title clearly summarises the main change: scheduled shell drift checks will run twice a month. |
✨ Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
State the approved 1st and 15th at 07:23 UTC schedule, keeping the source/PR validation requirement unchanged.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The background shared-site-shell comparison currently runs every Monday at 07:23 UTC. Run it on the 1st and 15th of each month at the same UTC time (
23 7 1,15 * *).Manual execution, the drift issue handling and the separate source/PR validation and Pages deployment workflow are unchanged.
Validation: both repository workflows parse with Ruby Psych; all cron fields are within range; before/after workflow objects match after excluding
on.schedule; the diff contains only the cron and its matchingdocs/SPEC.mdcadence sentence;git diff --checkpassed. No GitHub workflow was manually dispatched.Validation used an isolated workflow/instruction snapshot from the default-branch commit. CodeRabbit raised one minor issue: the specification still stated a weekly background check. The approved first/15th cadence is now documented in
docs/SPEC.md, retaining the source/PR validation requirement. A follow-up CodeRabbit review was blocked by execution approval; validation of the final documentation substitution passed locally.Automatic PR CI on the previous head
1a563c0a0698190ac54adaf3560cc6709563c553: 156 unit tests passed (24 deselected), manifests and runtime fingerprints passed. The unchanged shared-shell source check failed withShellError: footer column needs one heading and an accessible label; site hygiene and deployment were skipped. This PR does not alter the shell source or checker. Run: https://github.com/comphy-lab/talks/actions/runs/38634050585.Final head
57033489c6d00cd3d5886fe606587738c22d086d: automatic PR CI completed with 156 unit tests passed (24 deselected), manifests and runtime fingerprints passed. The unchanged shared-shell source check failed with the same footer error (ShellError: footer column needs one heading and an accessible label), and deployment was skipped. Run: https://github.com/comphy-lab/talks/actions/runs/38635541542. No CI rerun or manual dispatch was requested.