- Assurez-vous qu'il n'y a pas de demandes de PR existantes tentants de résoudre le problème mentionné;
- Les changements importants doivent être discutés dans le ticket;
- Développez dans une branche spécifique, pas sur la branche principale (2.5);
- Donnez un titre à votre PR qui respecte la spécification conventional commits;
- Donnez une description à votre PR et expliquez pourquoi nous devrions la merger;
- Ajouter autant que possible des tests;
Pour les tests fonctionnels, l'utilisation de Cypress nécessite d'installer NodeJS en version 18. Nous vous recommandons pour cela d'installer NodeJS via NVM.
Les actions ci-dessous nécessitent de disposer de PHP en version 8.3. Un exemple d'utilisation des commandes est disponible dans la section scripts du fichier composer.json.
- Exécuter Rector;
- Exécuter PHP-CS-Fixer;
- Exécuter PHPStan et ne pas ajouter d'erreur (dans la mesure du possible). Si la correction de l'erreur est trop complexe, l'ajouter dans la baseline;