Skip to content

fix(docs): drop the deleted per-repo deploy.yml gate reference (#293) - #8

Merged
github-actions[bot] merged 2 commits into
mainfrom
feat/drop-deploy-gate-ref
Sep 28, 2026
Merged

github-actions[bot] merged 2 commits into
mainfrom
feat/drop-deploy-gate-ref

Conversation

@aitrawog

@aitrawog aitrawog commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Fixes the R1 doc-vs-reality divergence opencharly/opencharly#293: the merged #286
AGENTS.md/README.md named .github/workflows/deploy.yml as the manifest gate. That
per-repo gate was retired org-wide by opencharly/.github#141. The merge gate is the
org-wide charly/pr-validator (required check validate / validate, defined in
opencharly/.github); this repo has no per-repo candy gate. No behavior change.

Refs #286, #293

How tested

$ gh api repos/opencharly/layer-nvidia/contents/.github/workflows --jq '[.[].name]|join(", ")'
tag-on-merge.yml

$ git grep -n deploy.yml            # repo-wide; the retired identifier
CHANGELOG/2026.271.1114.md:19:  commands (`charly box validate`, `.github/workflows/deploy.yml`); the edit rule

$ git diff --stat origin/main..HEAD
 AGENTS.md | 12 +++++-------
 README.md |  2 --
 2 files changed, 5 insertions(+), 9 deletions(-)

Rulebook compliance

Rule HOW / N/A
1 N/A — docs-only; edit via session worktree.
2 All git via git -C <abs>.
3 N/A — no Go.
4 N/A — docs-only.
5 N/A — no pin change.
6 N/A — no gitlink change.
7 N/A — no subrepo content edited.
8 N/A — no harness config change.
9 Issue #293 filed; no other session's work touched.
10 Read #293 + the live workflow listing before writing this body.
11 Campaign ledger maintained.
R1 The divergence (#293) is root-caused (.github#141) and fixed here.
R2 No surfaced failure parked; the version:-stamp migration is a separate non-.md leg on #293.
R3 N/A — no duplication.
R4 N/A — the fix states the real gate.
R5 The retired deploy.yml identifier is swept repo-wide (grep pasted).
R6 No force-push; no main push.
R7 Workflow listing + repo-wide grep + diff --stat pasted above.
R8/R9/R10 N/A — no build/runtime/agent-control/bed change.

Change classification

  • Class: documentation-only (AGENTS.md, README.md).
  • Gate: non-runtime standards (repo-wide grep + workflow listing).
  • Tier: documentation reviewed.

Agent: org-readme-agents · session ses_org-readme-agents
Assisted-by: opencode deepseek-v4.1-flash (documentation reviewed)

The #286 docs named .github/workflows/deploy.yml as the gate; retired org-wide by opencharly/.github#141. The merge gate is the org-wide charly/pr-validator (required check validate / validate). Refs #286, #293.
Prior replacement presented .github/workflows/ (a directory) as the gate. State the real merge gate: the org-wide charly/pr-validator (required check validate / validate, defined in opencharly/.github); this repo has no per-repo candy gate. Refs #286, #293.
@github-actions

Copy link
Copy Markdown

Review — PASS

Head SHA: c9b2bf13bcdbd5574d0e8a81957d9f28ece2ec9d (branch: feat/drop-deploy-gate-ref)

Security screen: PASS (docs-only; no secrets, no guardrail/rulebook edits, no scope mismatch, no embedded directives).
Checklist: All applicable rules PASS or NA (docs-only: no bed/R6/R8/R9/R10, no placement, no Go/ADE/SDD surface). Body ends with the required *Assisted-by:* trailer; tier documentation reviewed matches the all-.md diff; single atomic commit; the pasted workflow listing (tag-on-merge.yml only) is internally consistent with the diff removing the retired deploy.yml reference.
Comments: none (0-comment thread — nothing to disposition).

Verdict: PASS


ollama-cloud/deepseek-v4.1-flash — action-review.

View action run

@github-actions
github-actions Bot enabled auto-merge (squash) September 28, 2026 18:11
@github-actions

Copy link
Copy Markdown

Review — PASS

Head SHA: fb5f54255999708668ac12070a3a0fe8108342fd (branch: feat/drop-deploy-gate-ref)

Security screen: PASS
Checklist: All applicable rules PASS or NA (docs-only).

Comments: 1 considered — @github-actions[bot] PASS at c9b2bf13… → stale-superseded (it reviewed the first commit, not head fb5f5425…; its "single atomic commit" claim no longer matches the two-commit branch, but both are non-changelog docs commits that squash with no partial/split work pattern, so B15 is a note, not a block). No approve grants anything; re-derived independently — the current diff (AGENTS.md +5/-7, README.md +0/-2), the pasted workflow listing (tag-on-merge.yml only), the repo-wide grep (sole remaining hit is the historical CHANGELOG/2026.271.1114.md:19 entry — read, not mechanically deleted, per R5), and the git diff --stat (5 insertions, 9 deletions) are all internally consistent; body accounts for both files, ends with the required *Assisted-by:* trailer, and its documentation reviewed tier matches the all-.md diff.

Verdict: PASS


ollama-cloud/deepseek-v4.1-flash — action-review.

View action run

@github-actions
github-actions Bot merged commit eefa6a0 into main Sep 28, 2026
1 check passed
charly-auto-merge Bot added a commit that referenced this pull request Sep 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants