From 7a77534d36c256989a01c8bb4027deb3986b4b60 Mon Sep 17 00:00:00 2001 From: Nicolas Date: Fri, 18 Sep 2026 08:47:43 +0200 Subject: [PATCH] Simplifier le trigger de ci.yml (pull_request uniquement) 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 --- .forgejo/workflows/ci.yml | 3 --- spec/ci-cd.md | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/.forgejo/workflows/ci.yml b/.forgejo/workflows/ci.yml index 999ff65..9855477 100644 --- a/.forgejo/workflows/ci.yml +++ b/.forgejo/workflows/ci.yml @@ -4,9 +4,6 @@ on: pull_request: branches: - main - push: - branches-ignore: - - main jobs: build-lint-test: diff --git a/spec/ci-cd.md b/spec/ci-cd.md index 6f16231..81e7cab 100644 --- a/spec/ci-cd.md +++ b/spec/ci-cd.md @@ -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`