Résolue

Pull request sur un dépôt github

# Git # Tests unitaires

Bonjour Thibault,
J'ai besoin de contribuer à un projet sur github concernant ansible : https://github.com/netbox-community/ansible_modules/pull/1023

Ma Pull Request semble être acceptée mais on me demande de lancer des tests... et là, je suis complètement perdu... (pytest...)
Est ce que tu sais comment faire ?
Merci
P. Morry

Thibault houdon

Mentor

Bonjour Philippe,

En fait ta pull request bloque au niveau de la validation de la syntaxe (linting) avec Black, un outil qui sert à cela.

As-tu lancé le fichier ./hacking/black.sh comme rodvand t'as indiqué ? En lançant le script .sh dans ton terminal, Black va vérifier la syntaxe de ton code et t'indiquer s'il y a des changements à faire, ce qui semble être le cas ici car l'intégration continue (CI) ne passe pas :
fail black

Par la suite dans l'intégration continue, il y a effectivement des tests unitaires et d'intégration qui seront effectués :
Intégration continue

Si ces tests passent, ça signifie que ton code ne casse rien (en théorie ^^). Et ta pull request sera acceptée.

Tu n'es pas obligé de lancer ces tests de ton côté, même si tu peux effectivement le faire en exécutant la commande pytest directement à la racine du projet dans ton terminal.

Merci Thibault,
du coup j'ai installé black et j'ai refais un push de mon code et là en effet, automatiquement, Github à relancé un PR sur la branche dev du projet. C'est étonnant car j'ai fait la correction au sein de mon projet forké, mais la liaison c'est faite toute seule ! Cool !
Merci pour tes explications.
Philippe

Thibault houdon

Mentor

Salut Philippe !

En fait, ce qui se passe c'est que lorsque tu fais un push sur la branche de ton fork qui est liée à la pull request, GitHub met automatiquement à jour la pull request avec tes nouvelles modifications. C'est super pratique et ça évite de devoir créer de nouvelles pull requests à chaque fois que tu fais des modifications ;)

Inscris-toi

(c'est gratuit !)

Inscris-toi

Tu dois créer un compte pour participer aux discussions.

Créer un compte

Rechercher sur le site

Formulaire de contact

Inscris-toi à Docstring

Pour commencer ton apprentissage.

Tu as déjà un compte ? Connecte-toi.