Pull Request

Pull Request / Peer review

Chaque fonctionnalité développée dans une branche feat/* (voir Convention Git) est fusionnée dans develop via une Pull Request, revue par au moins un autre membre de l'équipe.

Template de description de PR

## Quoi
Résumé de la fonctionnalité / du correctif.

## Comment tester
Étapes pour vérifier le comportement en local.

## Liens
- Tâche Trello : #...
- Branche : feat/...

Exemple réel (PR #29)

Une vraie Pull Request fusionnée sur le dépôt, pour référence :

• Titre : animation légère sur les cards
• Branche : feat/animations → develop
• Description : « Ajout d'une animation légère sur les cards avec Tailwind Animations »
• Auteur : @Chidori06 — Relecteur demandé : @porte-christophe — Fusionnée par : @porte-christophe
• 2 commits, 15 ajouts / 4 suppressions sur 5 fichiers

🔗 Voir la PR #29 sur GitHub 🔗 Historique complet des commits sur main

⚠️ À noter pour la suite : plusieurs PR du dépôt (ex. #27, #30, #31, #32) n'ont pas de description remplie. L'objectif est de systématiser le template ci-dessus sur les prochaines PR.

Checklist avant d'ouvrir une PR

• La branche est à jour avec develop
• Le code compile et les tests passent (ng test)
• Pas de console.log ou de code commenté oublié
• Les erreurs API sont gérées (chargement, erreur, cas vide)
• L'affichage est responsive (mobile / tablette / desktop)
• Les commits respectent la convention (voir Convention Git)

Checklist de review (pour le relecteur)

• Le code répond au besoin décrit dans la PR
• Les composants sont réutilisables et correctement organisés
• Pas de duplication évidente avec un composant existant
• Le style respecte l'identité visuelle du projet (voir Maquettes)

Après validation

• Fusion dans develop (jamais directement dans main)
• Suppression de la branche feat/* fusionnée
• Mise à jour du statut de la tâche dans Trello