Éléments à considérer lors d'une Pull Request

Pour déterminer si une pull request devait être acceptée, nous nous basions sur les critères suivants :

Conditions d’acceptation d’une Pull Request

1. Respect du style de code

Le code soumis devait respecter la charte de style de l’équipe.
Voir la charte de style

---

2. Limite de fichiers modifiés

La pull request ne devait pas modifier plus de 5 fichiers (hors fichiers binaires).
Les fichiers binaires n’étaient pas pris en compte, car au début du projet, nous n’avions pas encore réussi à bien configurer le .gitignore pour les exclure.

Cette limite permettait de :

• Mieux isoler les tâches
• Faciliter les revues de code

---

3. Commentaires dans le code

Même si cela n’était pas obligatoire immédiatement, il était attendu que des commentaires pertinents soient ajoutés dans le code pour aider à la relecture.
Cette étape devait être complétée avant la fin du projet.

---

4. Relecture par l’équipe

Pour qu’une pull request soit acceptée :

• Au moins deux membres de l’équipe devaient l’avoir revue avec l’auteur.
• Les revues étaient principalement faites en appel Discord, afin de :
  • Faciliter la communication
  • Argumenter collectivement
  • S’assurer que les décisions prises étaient pertinentes

---