Simplifier le trigger de ci.yml (pull_request uniquement)
All checks were successful
CI / build-lint-test (pull_request) Successful in 44s

Le trigger push sur les branches non-main faisait tourner ci.yml deux
fois sur le meme commit des qu'une PR est ouverte (une fois pour le
push, une fois pour la pull_request) : deux statuts distincts pour un
seul resultat logique, et une exposition doublee aux flakys du runner
(observe : Setup pnpm a echoue une fois avec l'erreur text file busy,
une race condition transitoire juste apres l'installation de Node par
le runner - le run pull_request sur le meme commit avait reussi).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Nicolas 2026-09-18 08:47:43 +02:00
parent e2d7207237
commit 7a77534d36
2 changed files with 1 additions and 4 deletions

View file

@ -4,9 +4,6 @@ on:
pull_request:
branches:
- main
push:
branches-ignore:
- main
jobs:
build-lint-test:

View file

@ -6,7 +6,7 @@ Les workflows vivent dans `.forgejo/workflows/` (pas `.github/workflows/` — ce
### `ci.yml`
Déclenché sur `pull_request` vers `main` et `push` sur toute branche autre que `main`. Étapes : install (`--frozen-lockfile`) → `pnpm lint` → `pnpm build` → `pnpm test` → `pnpm docs:check`. Ne nécessite aucun secret (les dépendances internes entre packages passent par `workspace:*`, résolu localement).
Déclenché uniquement sur `pull_request` vers `main` (pas de trigger `push` séparé sur la branche source : il ferait tourner la même vérification deux fois sur le même commit dès qu'une PR est ouverte — deux statuts distincts pour un seul résultat logique, et une exposition doublée aux flakys du runner). Étapes : install (`--frozen-lockfile`) → `pnpm lint` → `pnpm build` → `pnpm test` → `pnpm docs:check`. Ne nécessite aucun secret (les dépendances internes entre packages passent par `workspace:*`, résolu localement).
### `release.yml`