-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Évite que la CI tourne sans PR #1788
Conversation
Bonjour, Dans https://github.com/openfisca/openfisca-france/pull/1786/files#diff-34819666b47069e1b67298b87d6fbd56a1b8bde32f43b5e0bf13e44c74427a65R4, je propose:
=> Conserver les tests sur tous les push, et sur les PR seulement à l'ouverture.
=> Test uniquement sur les PR, et lors du push sur master. Donc les tests ne tournent pas tant qu'on n'ouvre pas de PR. Je n'ai pas d'avis sur la question. La deuxième option est plus écologique car elle limite le nombre de CI. Personnellement je ne m'occupe pas des tests tant que je n'ai pas ouvert de PR. |
Bonjour Benoit ! |
27e1a07
to
40a8f63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
La branche est à mettre à jour, mais je pense que le fonctionnement apporté par la PR est le bon. La CI tourne en cas de PR (draft ou normale) et en cas de merge sur master (ce qui est une bonne pratique étant donné qu'on peut avoir un scénario où une branche pas à jour merge sur master peut entrainer l'échec d'un test)
Personnellement, j'apprécie avoir un retour de la CI lorsque je push. Je pense que cela peut réduire le bruit (et donc la perte de temps des reviewers) en évitant que des personnes n'ouvrent une PR et ne découvrent qu'à ce moment-là qu'elles ont des retours d'erreur automatique. Je trouverais regrettable que chaque PR commence par 2 jours de délai et 3 commits avant qu'elles ne soient réellement ouvertes pour revue. Je pense notamment aux personnes ayant une connaissance technique limitée et qui débute, et pour qui il peut être plus simple de faire un push que d'exécuter les tests localement. Pour ma part, je m'en sers pour paralléliser l'exécution des tests et le développement lorsque j'ai atteint un point important où je crois mon code stabilisé. Bien évidemment, pour un usage économe, cela suppose que les push ne soient pas faits à chaque commit. |
Très bien ! Dans ce cas je ferme cette PR. Merci pour ton retour @MattiSG ! |
Merci de contribuer à OpenFisca ! Effacez cette ligne ainsi que, pour chaque ligne ci-dessous, les cas ne correspondant pas à votre contribution :)
.github/workflows/workflow.yml
.Quelques conseils à prendre en compte :
setup.py
.CHANGELOG.md
.Et surtout, n'hésitez pas à demander de l'aide ! :)