Skip to content

docs: align chapter review checklist gates #125

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
jeresoftx merged 1 commit into from
Jul 22, 2026
Merged

docs: align chapter review checklist gates #125

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
28 changes: 24 additions & 4 deletions docs/checklist-revision-capitulo.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ pero el criterio humano decide qué queda aprobado (RFC-0001 §20).
Para revisar el curso completo como corte editorial, usa también
`docs/guia-revision-corte.md`.

Completar esta checklist no aprueba ni publica el capítulo. Solo reúne
evidencia para que Joel pueda decidir si el contenido puede avanzar a
`reviewed` o si necesita correcciones puntuales.

## Identificación

- Capítulo:
Expand All @@ -16,6 +20,26 @@ Para revisar el curso completo como corte editorial, usa también
- Revisor humano:
- Fecha de revisión:

## Evidencia mínima del corte

Antes de mover cualquier capítulo a `reviewed` o preparar una publicación
posterior, deben pasar las compuertas actuales:

- [ ] `node scripts/verify-course-links.mjs` pasa.
- [ ] `node scripts/verify-manifest-consistency.mjs` pasa.
- [ ] `node scripts/verify-cargo-examples.mjs` pasa.
- [ ] `node scripts/verify-chapter-anatomy.mjs` pasa.
- [ ] `node scripts/verify-review-packet-coverage.mjs` pasa.
- [ ] `node scripts/verify-editorial-status.mjs` pasa.
- [ ] `node scripts/verify-gate-sync.mjs` pasa.
- [ ] `cargo fmt --check` pasa.
- [ ] `cargo clippy --all-targets --all-features -- -D warnings` pasa.
- [ ] `cargo test --all-targets` pasa.
- [ ] `cargo test --doc` pasa.
- [ ] `cargo bench --all-targets` pasa.
- [ ] `git diff --check` pasa.
- [ ] El check remoto `rust` en GitHub Actions pasa.

## Claridad técnica

- [ ] El capítulo explica el problema antes de presentar servicios.
Expand All @@ -30,14 +54,10 @@ Para revisar el curso completo como corte editorial, usa también
- [ ] El código declara invariantes, límites y errores relevantes.
- [ ] No usa `unsafe`.
- [ ] No agrega dependencias externas sin justificación escrita.
- [ ] `cargo fmt --check` pasa.
- [ ] `cargo clippy --all-targets --all-features -- -D warnings` pasa.

## Pruebas y benchmarks

- [ ] Hay pruebas para reglas y casos límite.
- [ ] `cargo test --all-targets` pasa.
- [ ] `cargo test --doc` pasa.
- [ ] El capítulo declara si benchmark aplica.
- [ ] Los resultados no prometen costos ni rendimiento de producción.

Expand Down
Loading