Skip to content

ci(policy): repin the organization commit-lint workflow - #235

Merged
ss-o merged 1 commit into
mainfrom
feature-774
Oct 9, 2026
Merged

ss-o merged 1 commit into
mainfrom
feature-774

Conversation

@ss-o

@ss-o ss-o commented Oct 9, 2026

Copy link
Copy Markdown
Member

Summary

Refs z-shell/.github#774

Verification

Agent handoff

No handoff needed.

@ss-o ss-o added area:ci Continuous integration or GitHub Actions work. type:maintenance Non-feature maintenance, cleanup, or org work. labels Oct 9, 2026
@ss-o ss-o self-assigned this Oct 9, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying src with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1260470
Status: ✅  Deploy successful!
Preview URL: https://8ea3c484.zi-src.pages.dev
Branch Preview URL: https://feature-774.zi-src.pages.dev

View logs

@ss-o ss-o left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fallback review under ADR-0026: Copilot request not registered on 1260470

Findings

No actionable findings.

Scope: the complete diff of 1260470 against main at 2c5eaac: one file, .github/workflows/commit-lint.yml, with the reusable-workflow pin moved from 64a427d to 6fac3d4 and the concurrency comment rewritten.

Important

Sole-maintainer exception under ADR-0035. The maintainer declared that no second human is available and elected the exception for this pull request at 1260470. A Copilot review was requested on 2026-10-09 at about 12:53Z and no review_requested event for Copilot followed; the cause is unconfirmed. A separate read-only review of the same head ran in a different reviewing session, as ADR-0035 decision 3 requires. Additional automated scrutiny is not a second human.

Note

This review is by the pull request's author. It does not approve deployment or release.

Separate read-only review

Head confirmed with the diff one file, .github/workflows/commit-lint.yml, four lines changed. No actionable findings. Pin: 6fac3d4 is reachable from z-shell/.github main and is the latest commit touching commit-lint.yml there; it differs from 64a427d only by the removed concurrency group, and has no concurrency key. Caller compatibility: both called-workflow inputs are optional and the caller passes none; permissions fit within contents: read. Supply chain: full-SHA pin on main, actions/checkout pinned by SHA, no pull_request_target, no secrets. Deployment: no push-to-main build or deploy reads this file. Limits: the called workflow was compared by diff, not executed locally.

Validation

  • Passed: all 15 hosted checks on 1260470, including the four required Policy checks, which ran against 6fac3d4.
  • Copilot: requested once on 2026-10-09 at about 12:53Z; no review_requested event for Copilot followed. Cause unconfirmed.
Checklist verdicts and supporting evidence
  • Pin: correct. 6fac3d4 is a full SHA on z-shell/.github main and the latest commit there that changes commit-lint.yml; 64a427d is on no branch (z-shell/.github#774). The two versions differ only by the removed concurrency group.
  • Compatibility: unchanged. The caller's trigger, inputs and contents: read permission are as before; the called workflow's inputs are optional.
  • Comment: accurate. Neither the caller nor the called workflow declares a concurrency group at 6fac3d4.

Limits and follow-up

  • Recovery: revert the squash commit; the previous caller is restorable from main at 2c5eaac.
  • The cause of the startup failure on 64a427d is inferred from its absence from every branch, not confirmed; tracked in z-shell/.github#774.

@ss-o
ss-o merged commit 3e17623 into main Oct 9, 2026
17 of 19 checks passed
@ss-o
ss-o deleted the feature-774 branch October 9, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ci Continuous integration or GitHub Actions work. type:maintenance Non-feature maintenance, cleanup, or org work.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant