From c9b2bf13bcdbd5574d0e8a81957d9f28ece2ec9d Mon Sep 17 00:00:00 2001 From: atrawog Date: Mon, 28 Sep 2026 20:05:58 +0200 Subject: [PATCH 1/2] fix(docs): drop the deleted per-repo deploy.yml gate reference (#293) 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. --- AGENTS.md | 12 +++++------- README.md | 3 +-- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index b3c0e31..47cfe03 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -10,7 +10,7 @@ Canonical files: - `charly.yml` — the `nvidia:` candy entity and the `nvidia-skill:` skill entity. - `CHANGELOG/` — per-CalVer release history; read it before changing baked checks. -- `.github/workflows/deploy.yml` — the manifest gate. +- `.github/workflows/` — the org-wide `charly/pr-validator` gate; there is no per-repo candy gate. - `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge. - `README.md` — user overview only; never agent guidance. @@ -26,12 +26,10 @@ Canonical files: ## Build / validate / test -- `charly box validate` at the repo root — the same structural gate CI runs: the - manifest must parse and validate at the pinned charly. The CI pin lives in - `.github/workflows/deploy.yml`; keep the `version:` schema stamp within the - pinned charly's supported range (do not migrate the stamp past the pin). -- `.github/workflows/deploy.yml` — builds the pinned charly from a CI-time - checkout and runs `charly box validate`. This is the merge gate. +- `charly box validate` at the repo root — the structural check: the manifest + must parse and validate at the installed charly. +- The merge gate is the org-wide `charly/pr-validator` (required check + `validate / validate`); there is no per-repo candy gate. - The candy's `plan:` `check:` steps are the functional evidence; they must stay valid on every distro arm they run on. Where a check only applies to one distro family (e.g. a Fedora-only DNF path), scope it in the command itself — diff --git a/README.md b/README.md index 98601c8..138fd36 100644 --- a/README.md +++ b/README.md @@ -59,8 +59,7 @@ example `python-ml`, `jupyter`, `ollama`, and `comfyui`. `LD_LIBRARY_PATH` environment, the Vulkan ICD wiring `plan:`, the `plan:` checks, and the embedded `skill:` entity. - `CHANGELOG/` — per-CalVer release notes. -- `.github/workflows/deploy.yml` — builds the pinned charly and runs - `charly box validate` on the manifest (the merge gate). +- `.github/workflows/` — the org-wide `charly/pr-validator` gate; no per-repo candy gate. - `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge. - `README.md` — this user overview. From fb5f54255999708668ac12070a3a0fe8108342fd Mon Sep 17 00:00:00 2001 From: atrawog Date: Mon, 28 Sep 2026 20:10:10 +0200 Subject: [PATCH 2/2] fix(docs): state the org-wide pr-validator gate accurately (#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. --- AGENTS.md | 6 +++--- README.md | 1 - 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 47cfe03..38ba039 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -10,7 +10,6 @@ Canonical files: - `charly.yml` — the `nvidia:` candy entity and the `nvidia-skill:` skill entity. - `CHANGELOG/` — per-CalVer release history; read it before changing baked checks. -- `.github/workflows/` — the org-wide `charly/pr-validator` gate; there is no per-repo candy gate. - `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge. - `README.md` — user overview only; never agent guidance. @@ -28,8 +27,9 @@ Canonical files: - `charly box validate` at the repo root — the structural check: the manifest must parse and validate at the installed charly. -- The merge gate is the org-wide `charly/pr-validator` (required check - `validate / validate`); there is no per-repo candy gate. +- 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. - The candy's `plan:` `check:` steps are the functional evidence; they must stay valid on every distro arm they run on. Where a check only applies to one distro family (e.g. a Fedora-only DNF path), scope it in the command itself — diff --git a/README.md b/README.md index 138fd36..0a6ddc6 100644 --- a/README.md +++ b/README.md @@ -59,7 +59,6 @@ example `python-ml`, `jupyter`, `ollama`, and `comfyui`. `LD_LIBRARY_PATH` environment, the Vulkan ICD wiring `plan:`, the `plan:` checks, and the embedded `skill:` entity. - `CHANGELOG/` — per-CalVer release notes. -- `.github/workflows/` — the org-wide `charly/pr-validator` gate; no per-repo candy gate. - `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge. - `README.md` — this user overview.