Skip to content

🔧 chore(ci): faire tourner la CI sur main aussi - #74

Merged
jefBinomed merged 1 commit into
developfrom
chore/ci-main-triggers
Aug 13, 2026
Merged

🔧 chore(ci): faire tourner la CI sur main aussi#74
jefBinomed merged 1 commit into
developfrom
chore/ci-main-triggers

Conversation

@jefBinomed

Copy link
Copy Markdown
Collaborator

Fait tourner la CI sur main, pas seulement sur develop.

Le constat

En examinant les PR Dependabot ouvertes (#68, #69, #70), toutes ciblant main :

$ gh pr checks 68
no checks reported on the 'dependabot/npm_and_yarn/postcss-8.5.25' branch

Le workflow ne s'abonnait qu'à develop :

on:
  push:         { branches: [develop] }
  pull_request: { branches: [develop] }

main est la branche par défaut du dépôt — celle que voient les visiteurs, et celle que Dependabot cible pour ses mises à jour de sécurité. Toute PR vers main se mergeait donc sans qu'aucun test, lint, typecheck ou audit ne tourne.

Correctif

  • push et pull_request couvrent désormais develop et main ;
  • workflow renommé « CI — Node 24 » : l'intitulé « Develop CI » était devenu faux.

Le nom de fichier reste develop.yml. Il est référencé par le badge du README, par CLAUDE.md et par plusieurs documents de docs/improvements/ ; le renommer casserait aussi les liens vers les runs existants. Un commentaire en tête du fichier explique l'écart.

Points vérifiés avant de pousser

  • docs-deploy est déjà conditionné à github.ref == 'refs/heads/develop' : il skippe sur main, le déploiement des docs est inchangé.
  • Aucune protection de branche n'est configurée (main et develop répondent toutes deux 404 Branch not protected), donc le renommage du workflow ne casse aucune règle de checks requis.
  • Le YAML est valide et les 8 jobs sont bien chargés.

Suite

Une fois cette PR mergée, develop → main apportera à main un workflow déjà armé, en même temps que les correctifs de sécurité — ce qui rendra les trois PR Dependabot caduques.

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.
@jefBinomed
jefBinomed merged commit 6d0b1da into develop Aug 13, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant