@yanis.kocier ce serait possible que tu fasse un petit texte/ebauche de sujet pour qu'on comprenne bien la direction vers laquelle on va ?
Pour le moment te fait pas chier a render le truc, juste le text suffira :)
( sur une autre branch et pr stp )
je veux bien review, mais si deja c'est pas tres clair c'est embetant

Hello, @yanis.kocier ce serait possible que tu fasse un petit texte/ebauche de sujet pour qu'on comprenne bien la direction vers laquelle on va ? Pour le moment te fait pas chier a render le truc, juste le text suffira :) ( sur une autre branch et pr stp ) je veux bien review, mais si deja c'est pas tres clair c'est embetant

ClementForget (Migrated from git.la-banquise.fr) reviewed

2025-10-13 13:29:05 +02:00

ClementForget (Migrated from git.la-banquise.fr) left a comment

No description provided.

Hello, j'ai plusieurs points qui ne me vont pas ici. **Disclaimer : Je peux parraitre "dur" mais je veux juste faire des retours utiles** Ce que je vais dire ici sont mes points globaux, pour le reste cf le reste de la review ## But Pour commencer je me pose la question de ce qu'on veux vraiment leurs montrer. Dans ce que je vois, on veux leurs faire faire des socket mais en meme temps il a y a pas mal de bordel qui sert pas a grand choses selon moi. Maintenant @yanis.kocier est ce que le but etait de les faire aussi coder dans connexion.py ? ## Forme Sur la forme aussi j'ai des retours. Ya bcp de choses qui ressemble a du vibe codding @yanis.kocier Donc soit on fait un truc full en Fr (ce dont je suis pas fan mais je peux comrendre) soit en anglais, mais pas les deux. Il faudrais aussi raccroucir les noms de fonctions. Pour conclure ya encore du taff, j'en discutais avec Arthur, si jamais il est pas pret on fera le sujet python pour la prochaine JI. Donc vaux mieux faire un truc propre, et le proposer plus tard. @david.cozariuc je sais qu'une bonne partie de ce code n'est pas le tiens, donc tkt :)

david.cozariuc commented

2025-10-16 06:47:43 +02:00

(Migrated from git.la-banquise.fr)

@ClementForget

Merci pour le review, tu as été très clair sur ce qu'il n'y allait pas.

C'était plus l'idée d'utiliser des modules pour le client que je voulais mettre en avant dans ce PR.

Je n'avais d'ailleurs pas utilisé de vibe coding. Les type hints, docstrings, comments: je les rajoute par habitude.

Au final on utilisera la branche #3 et pas celle ci.

@ClementForget Merci pour le review, tu as été très clair sur ce qu'il n'y allait pas. C'était plus l'idée d'utiliser des modules pour le client que je voulais mettre en avant dans ce PR. Je n'avais d'ailleurs pas utilisé de vibe coding. Les type hints, docstrings, comments: je les rajoute par habitude. Au final on utilisera la branche #3 et pas celle ci.