- Omnipotent, omniscient, omnipresent
- https://social.mtdv.me/Laodeus
-
"Discretion is not the better part of biography... "
Thomas Carlyle - Joined on
2025-04-23
Pourquoi le retour est il un string? Pourquoi pas un boolean? Y'a til plus d'information necessaire au retour que effacer ou non effacer?
Pourquoi passer un spell entier quand on gagner de la memoire en ne passant que l'id du spell, et des parametres optionnel pour ce qui doit etre modifier?
Peux-tu vérifier si ces imports sont vraiment nécessaires ? En général, on évite les imports inutiles pour garder le code propre
Tu veux garder l'entieretée de l'objet User en en memoire pendant ton utilisation de cette entité? Une simple reference ne serait pas mieux?
Penses-tu qu'il soit souhaitable qu'un code externe puisse modifier l'Id d'un Spell après sa création l'entité ne devrais pas etre private?
Pour moi, c'est OK.
Je ne sais pas valider. Donc je te. Laisse merger.
Pour. Les nom de branche, ouep, continue, juste, fait correspondre le. Nom. De. La branche avec la feature que tu implémente.
Genre, feature/kickstart-project plutôt que feature/1
Une classe vide ne devrais pas éxister dans une pr, si elle est vide, c'est que soit, elle ne devrais pas se trouver dans la pr, soit la feature qu'elle doit implémenter n'exisqte pas encore.
Pour quek raison ton infrastructure ne reference pas ton domaine? Comment comptais tu implémenter tes repos sans reference au domaine dans ton infra?
C'est une api? Quel est l'interet d'ouvrir le browser automatiquement ?
Le nom de la branche feature/1 suit les precepte du gitflow, mais est tres peu pertinent
Les noms des commit sont assez minimaux.
Edit config : En quoi les config ont été modifié? Fix : qu'est ce qui as été fixé?
De plus, regarde le principe de commit atomique. ce…