Repository navigation
Conversation
- alembic-v1-check.yml: the folded pytest command becomes one line. - pr.yml: split apt-get update and the Redis install into two steps. - sync-cloud-run-secrets.yml: the master-only gate becomes require_git_ref.sh, tested, and checked before authentication. - cleanup_cloud_run_images.sh: add tests with a fake gcloud. - deploy_observability_collector.sh: group the GITHUB_OUTPUT writes (shellcheck SC2129); behaviour unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The multi-line baseline drops from 10 to 7 (the deploy pipeline remains). The untested-script and shellcheck baselines are empty, so api.ci.workflow-scripts-tested and api.ci.shell-scripts-pass-shellcheck move from ratcheted to enforced. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Once a CI-script rule is enforced it has no baseline, so its findings were reported as [registry]. Fall back to the rule whose id ends in the check's name (<repo>.ci.multiline-run-is-script, ...). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
anth-volk
added this pull request to stack #3888
October 12, 2026 20:30
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## rules-followups #3895 +/- ##
================================================
Coverage 87.56% 87.56%
================================================
Files 198 198
Lines 12211 12211
Branches 2159 2159
================================================
Hits 10692 10692
Misses 922 922
Partials 597 597 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This branch was successfully deployed
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.
Fixes #3893
Summary
Phase 1 step 1.7, part 1. Stacked on #3890; the deploy-pipeline part is the next PR in the stack.
alembic-v1-check.yml: the folded pytest command is now one line.pr.yml:apt-get updateand the Redis install are now two single-line steps.sync-cloud-run-secrets.yml: the master-only gate is a new tested script,require_git_ref.sh. The existing workflow test now checks that it runs before authentication.cleanup_cloud_run_images.shgets tests with a fakegcloud: keep the newest N, dry run, within the limit, and failed deletes.deploy_observability_collector.sh: theGITHUB_OUTPUTwrites are grouped (shellcheck SC2129). Behaviour is unchanged.api.ci.workflow-scripts-testedandapi.ci.shell-scripts-pass-shellcheckmove from ratcheted to enforced.ci-scriptsguard still attributes its findings to that rule rather than[registry].scripts/rules/stays byte-identical with sim's copy.Testing
tests/unit/test_workflow_guard_scripts.py: 6 pass. The related workflow and deploy-script tests pass too (139).python scripts/run_quality_guards.py: all three guards pass.guard --base-ref rules-followupspasses.python -m unittest discover -s scripts/rules/tests -t .: 75 pass.ruff format/ruff checkand actionlint are clean. All scripts pass shellcheck.🤖 Generated with Claude Code