Repository navigation
Conversation
Follow-up review pass — corrections appliedA further independent review of this PR found and I applied:
Verification after the correction
One thing to do during rolloutThis PR renames the PR-facing job, so if the repository's branch protection lists the old job name as a required status check, update it at merge time or pull requests will sit waiting for a check that never reports. The infrastructure PR documents this. Workflows in this repository remain the single place these pipelines are defined; runners are provisioned by the infrastructure PR and must be applied before merging. |
Second review pass — findings appliedAn independent review of this PR (and its siblings) found three issues, all fixed here:
Verification after the correctionsBoth workflow heads parse; the publication checkout expression and the concurrency group were asserted directly, and the job guards still route Not verified here: real runner delivery, registry access, and whether Runner provisioning must be applied from the infrastructure PR before this workflow routing is merged. |
Changes
Evidence and limitations
Parent verified all nine workflows against the GitHub Actions schema and exercised 24 actual version/tag Bash and Git consumer cases using file-only remotes, plus canonical-leading-zero rejection cases for both example/MyPics workflows. Independent app review round two scored 9 with no findings. No product image build or live-cluster/CNI test was performed.
Rollout dependency
Do not merge app routing before infrastructure PR humansoftware/self-host-saas-k3s#23 and human cutover PR humansoftware/human_infra#24 are merged and the infrastructure is applied. New-label CI may queue until that deployment. This PR does not merge or apply cluster changes.