Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
github-actions merged 2 commits into from
Sep 28, 2026
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c9b2bf1
fix(docs): drop the deleted per-repo deploy.yml gate reference (#293)
atrawog Sep 28, 2026
fb5f542
fix(docs): state the org-wide pr-validator gate accurately (#293)
atrawog Sep 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 5 additions & 7 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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/deploy.yml` — the manifest gate.
- `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge.
- `README.md` — user overview only; never agent guidance.

Expand All @@ -26,12 +25,11 @@ 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`, 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 —
Expand Down
2 changes: 0 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +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/deploy.yml` — builds the pinned charly and runs
`charly box validate` on the manifest (the merge gate).
- `.github/workflows/tag-on-merge.yml` — CalVer tag + `CHANGELOG/` on merge.
- `README.md` — this user overview.

Expand Down
Loading