Skip to content

ci: run the build on pull requests to the default branch - #62

Merged
phmatray merged 2 commits into
devfrom
ci/fix-pr-trigger
Jul 25, 2026
Merged

ci: run the build on pull requests to the default branch#62
phmatray merged 2 commits into
devfrom
ci/fix-pr-trigger

Conversation

@phmatray

Copy link
Copy Markdown
Contributor

La CI de ce depot ne s'executait sur aucune pull request

La branche par defaut est dev, mais main.yml filtrait ses PR sur - "main" # Trigger the workflow on pull requests to the main branch.

GitHub n'evalue le declencheur pull_request que si la branche cible correspond au filtre : aucune PR vers la branche par defaut ne declenchait donc de build. Consequences mesurees :

  • aucun check ne gardait les merges sur ce depot ;
  • l'automerge Renovate ne pouvait jamais s'armer (il attend un check vert), d'ou l'accumulation de PR.

Le filtre de branche est retire de pull_request (convention du portfolio : une PR vers n'importe quelle branche declenche le build). push est laisse intact : le repointer ferait tourner la CI sur la branche par defaut, ou un echec preexistant s'afficherait en rouge.

Cette PR est son propre test : le check qui apparait ci-dessous est la preuve que le declencheur fonctionne. Merge uniquement s'il est vert.

@phmatray

Copy link
Copy Markdown
Contributor Author

Le declencheur est repare — et il revele un build casse

La correction du filtre pull_request fonctionne : un check s'execute enfin sur ce depot. Il est rouge, et il l'etait deja avant cette PR — simplement, personne ne pouvait le voir.

NU1902AngleSharp 1.4.0 vulnerable (GHSA-pgww-w46g-26qg), avec l'audit NuGet en erreur. Correctif pousse sur cette branche (CPM 1.4.0 -> 1.5.2), verifie en local : l'erreur disparait.

Le build reste rouge pour une raison independante : l'app demo demo/HtmxAppServer a des erreurs de parsing Razor (CS0103 sur div, foreach, path — signature d'un @ parasite ou d'une balise non fermee). Preexistant, jamais vu car la CI ne se declenchait sur aucune PR.

PR laissee ouverte volontairement : la regle du portfolio est de ne merger que du vert. Reparer le build est un travail distinct de la reparation du declencheur.

@phmatray
phmatray merged commit 4963a08 into dev Jul 25, 2026
5 checks passed
@phmatray
phmatray deleted the ci/fix-pr-trigger branch July 25, 2026 23:34
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