Fix/supprime redondance readme #3
Conversation
| @@ -1,8 +1,8 @@ | |||
| name: Audit Performance | |||
| on: | |||
| push: | |||
| branches: | |||
| - main | |||
| paths-ignore: | |||
| @@ -1,8 +1,8 @@ | |||
| name: Audit Performance | |||
| on: | |||
| push: | |||
| branches: | |||
There was a problem hiding this comment.
On ne peut pas laisser la contrainte de branche quand on set un paths-ignore ?
There was a problem hiding this comment.
Si, on peut garder des contraintes de branche (cf différents commits). Est ce que tu souhaites déclencher la CI sur la branche principale uniquement, ou la branche principale et les PR ?
On peut déclarer un tableau du type: [main, pull_request]
There was a problem hiding this comment.
en fait, pour tout les tests audits et e2e, je préfère pour le moment ne les lancer que sur la branche main, pour :
- ne pas les lancer trop souvent, c'est très consommateur de ressource
- ne pas obliger pour le moment les personnes souhaitant faire des PR à avoir des tests nickels ...
There was a problem hiding this comment.
Ok, ça marche, je fais le changement.
There was a problem hiding this comment.
C'est quoi le style de la maison, je squash tous les commits de CI pour nettoyer, ou toute la branche ? :)
There was a problem hiding this comment.
Pas encore de style maison en fait :) Mais en ce qui me concerne, j'aime bien voir la logique des commits plutôt qu'un seul gros paquet. Je lance souvent des rebases interactifs pour remettre de l'ordre, mais en tout cas je ne fait que rarement qu'un seul commit. Sauf pour du hotfix, mais cela n'a pas de sens à mon avis sur un branche de feature.
FanchGadjo
force-pushed
the
fix/supprime-redondance-readme
branch
from
6f3d905
to
9eacaab
Compare
Description
Corrige une redondance dans le readme. Ajoute l'exclusion des readme dans la CI. J'ai préféré ne pas exclure tous les fichiers markdown pour l'instant.
Issue liée
Pas d'issue
ToDo list
Checklist
RFRa été ajouté indiquant aux autres contributeurs qu'elle est selon vous mergable