From 5e5b609c8f77b5bb637b7fc010d9987ce19a683e Mon Sep 17 00:00:00 2001 From: jefbinomed Date: Thu, 13 Aug 2026 14:18:11 +0200 Subject: [PATCH] =?UTF-8?q?=F0=9F=94=A7=20chore(ci):=20faire=20tourner=20l?= =?UTF-8?q?a=20CI=20sur=20main=20aussi?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Aucune PR ciblant `main` ne déclenchait le moindre contrôle : le workflow ne s'abonnait qu'à `develop`. Les trois PR Dependabot ouvertes (#68, #69, #70), qui visent `main`, affichent littéralement « no checks reported » — et `main` est la branche par défaut du dépôt, donc celle où atterrissent les mises à jour de sécurité. - `push` et `pull_request` couvrent désormais `develop` et `main` ; - workflow renommé « CI — Node 24 », l'intitulé « Develop CI » étant devenu faux. Le nom de FICHIER reste `develop.yml` : il est référencé par le badge du README, CLAUDE.md et plusieurs docs d'amélioration, et le renommer casserait les liens vers les runs existants ; - `docs-deploy` est déjà conditionné à `refs/heads/develop`, il skippe donc sur `main` sans rien changer au déploiement des docs. Aucune protection de branche n'est configurée sur le dépôt, donc le renommage du workflow ne casse aucune règle de checks requis. --- .github/workflows/develop.yml | 9 ++++++++- CLAUDE.md | 5 +++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/develop.yml b/.github/workflows/develop.yml index 2fdab61..f093b1d 100644 --- a/.github/workflows/develop.yml +++ b/.github/workflows/develop.yml @@ -1,12 +1,19 @@ -name: Develop CI — Node 24 +# Nom de fichier historique (`develop.yml`) : le workflow couvre aussi `main` +# depuis qu'on a constaté que les PR vers la branche par défaut ne déclenchaient +# strictement aucun contrôle — les PR Dependabot #68/#69/#70 affichaient +# « no checks reported ». Renommer le fichier casserait les liens existants vers +# les runs, on garde donc le nom et on corrige l'intitulé. +name: CI — Node 24 on: push: branches: - develop + - main pull_request: branches: - develop + - main concurrency: group: ${{ github.workflow }}-${{ github.ref }} diff --git a/CLAUDE.md b/CLAUDE.md index bc9cb31..9deb6f0 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -52,6 +52,11 @@ là-dessus » à partir d'un échec local, vérifier que le contrôle en questio `.github/workflows/develop.yml` — `npm audit` n'en avait aucun jusqu'à la mise en place du job `audit`, et un échec local ne rendait donc pas la CI rouge. +Le workflow s'appelle `develop.yml` pour des raisons historiques mais couvre **`develop` ET `main`** +(push et pull_request). Avant, aucune PR vers `main` — la branche par défaut, cible des PR +Dependabot — ne déclenchait le moindre contrôle. Seul `docs-deploy` reste conditionné à +`refs/heads/develop` et skippe sur `main`. + ## Contrat iso-fonctionnel `docs/migration/golden-path.md` fait foi : le comportement utilisateur décrit là est le contrat,