Conversation
Mopolo
force-pushed
the
doc-adr-doctrine-glossaire
branch
from
a379b18 to
f20528c
Compare
Mopolo
force-pushed
the
doc-adr-doctrine-glossaire
branch
from
8f2aac0 to
5e0a8ec
Compare
Mopolo
changed the title
|
J'ai repris la PR pour y mettre plus de forme. Je vous présente ça au prochain point mensuel. |
|
Les ADRs sont à jour suite à notre discussion de ce midi. Je vous laisse à votre relecture ! |
| | Domaine | Français | `Inscription`, `Facture`, `Adresse`, `StatutCommande` | Le code doit parler la langue du métier | | ||
| | Infrastructure | Hybride | `AntenneRepository`, `CommandeAdapter`, `FactureSender` | Les mots métier sont en français puisque le métier de l'asso est en français, et les mots techniques sont en anglais, pas de raison de les traduire | | ||
| | Code technique | Anglais | `FileUploader`, `HttpClient`, `EventDispatcher` | Les mots du métier de dev sont toujours en anglais, pas de raison de les traduire | | ||
| | Commentaires | Français | `// Vérification du numéro de facture` | Pour rester raccord avec le code métier | | ||
| | Tests Behat | Hybride | `When I press "Sauvegarder"` | Les phrases Behat proviennent d'une librairie et sont déjà en anglais, l'interface est en français | |
There was a problem hiding this comment.
Salut, je me permets d'intervenir sur ce sujet qui me tiens à coeur. Ça fait 6 ans maintenant que dans ma boite on code en français (à mon initiative). Ce tableau reflète bien l'équilibre qu'on a trouvé dans notre équipe.
Je rajouterais une note pour ne pas utiliser d'accents dans le code tout de même. On avait commencé en en mettant partout mais au final ça nous à posé des problèmes avec plusieurs outils :
- Psalm n'arrive plus à référencer le type d'un argument si le type est définit en docblock d'une méthode (je ne sais pas si PHPStan a le même problème)
- Certaines parties de Doctrine ne les interprète pas correctement tel que le Query Builder (mais un usage de base de Doctrine fonctionne)
- On avait eu des problèmes au niveau SQL (mais je ne sais pas si vous comptez utiliser des accents dans les noms de table/colonne)
- Avec Git en fonction de la config des ordi et de l'OS il affiche les codes unicode au lieu des caractères dans le terminal. En soit ça fonctionne mais c'est pas très user friendly et pourrait faire peur à un nluveau contributeur novice.
En tout cas ça fait plaisir de voir la pratique se diffuser 😍
|
|
||
| ### Négatives | ||
|
|
||
| - Les habitudes ont la vie dure, et écrire du code en français amènera toujours son lot de débat |
There was a problem hiding this comment.
Vu qu'on a quelques années de recul avec mon équipe, si vous avez déjà des interrogations peut être que je peux aider 🙂
3 participants
Cette PR ajoute un dossier d'ADR ainsi que 2 premières.
Les documents sont plus agréables à lire en version rendues :