Poser une CI Forgejo Actions sur ce depot #24

Open
opened 2026-08-16 18:28:49 +00:00 by maximus · 0 comments
Owner

Constat structurel remonte par la review de la pile #18-#23 : ce depot n'a aucune CI.
Ni .forgejo/workflows/ ni .github/.

Consequence concrete sur la vague qui vient d'etre mergee : aucun des six maillons, pas meme
le premier contre main, n'a jamais ete verifie par une machine. Les runs locaux ont ete la
seule barriere — et ils ont tenu parce qu'un humain et des agents les ont lances a la main a
chaque etape, ce qui n'est pas une garantie reproductible.

Le depot a pourtant de quoi faire tourner une CI utile : npm test couvre 251 cas, runtime
0-dependance, node:22-alpine.

Criteres d'acceptation

  • .forgejo/workflows/ci.yml : npm ci puis npm test sur pull_request et sur push vers main
  • Le filtre de branche ne se limite PAS a pull_request: branches: [main] — sinon les maillons intermediaires d'une pile ne sont jamais batis (c'est le piege documente dans le skill merge-stack)
  • Un build de l'image Docker, pour attraper un COPY incomplet : c'est exactement le mode d'echec que #11 a du corriger a la main (metrics.js absent du COPY aurait plante le conteneur au redeploy, sans qu'aucun test ne le voie)
  • Le runner Forgejo est enregistre au niveau de ce depot (voir le skill diag-forgejo-ci : un runner enregistre ailleurs ignore les jobs en silence)
  • Une PR de test montre le job vert

Ref : skill diag-forgejo-ci.

Constat structurel remonte par la review de la pile #18-#23 : **ce depot n'a aucune CI**. Ni `.forgejo/workflows/` ni `.github/`. Consequence concrete sur la vague qui vient d'etre mergee : aucun des six maillons, pas meme le premier contre `main`, n'a jamais ete verifie par une machine. Les runs locaux ont ete la seule barriere — et ils ont tenu parce qu'un humain et des agents les ont lances a la main a chaque etape, ce qui n'est pas une garantie reproductible. Le depot a pourtant de quoi faire tourner une CI utile : `npm test` couvre 251 cas, runtime 0-dependance, `node:22-alpine`. ## Criteres d'acceptation - [ ] `.forgejo/workflows/ci.yml` : `npm ci` puis `npm test` sur `pull_request` **et** sur `push` vers `main` - [ ] Le filtre de branche ne se limite PAS a `pull_request: branches: [main]` — sinon les maillons intermediaires d'une pile ne sont jamais batis (c'est le piege documente dans le skill `merge-stack`) - [ ] Un build de l'image Docker, pour attraper un `COPY` incomplet : c'est exactement le mode d'echec que #11 a du corriger a la main (`metrics.js` absent du COPY aurait plante le conteneur au redeploy, sans qu'aucun test ne le voie) - [ ] Le runner Forgejo est enregistre au niveau de ce depot (voir le skill `diag-forgejo-ci` : un runner enregistre ailleurs ignore les jobs en silence) - [ ] Une PR de test montre le job vert Ref : skill `diag-forgejo-ci`.
maximus added the
status:ready
type:infra
source:human
labels 2026-08-16 18:28:49 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: maximus/vps-health-api#24
No description provided.