Skip to content
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

Suggestion pour accélération du passage des tests avant le merge d'une PR #2356

Open
DorineLam opened this issue Sep 26, 2024 · 2 comments

Comments

@DorineLam
Copy link
Contributor

Hello hello !

Je suis le fan numéro un d'OpenFisca, et j'ai une suggestion d'amélioration du passage des tests pour que les contributeurs gagnent du temps dans un cas spécifique.

Qu'ai-je fait ?

J'ai créé une PR, fait mes modifs, attendus plusieurs fois que les tests passent car j'avais fait une erreur... Et puis finalement tout est passé ... Mais un autre fan d'OpenFisca a été plus rapide que moi et a mergé sa PR avant la mienne. Résultat : J'ai du changer le changelog et le setup (normal) mais j'ai du aussi attendre que tous les tests repassent ...

Ce dont je rêverai :

Ne pas avoir à repasser tous les tests si seuls les fichiers de changelog et de setup ont changé !

@DorineLam
Copy link
Contributor Author

@sandcha pour info, voici l'issue suite à notre discussion :)

@benjello
Copy link
Member

Le fichier setup.py contient les dépendances. Si on les change, il est essentiel de vérifier qu'elle ne casse rien en aval donc que les tests tournent.

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

No branches or pull requests

2 participants