J'ai laissé 21 coms en reviewant 290 lignes Python dans la PR de ma N+1

// d#velope the code of something

encodage, MR bloquée

Le 22 juin 2022 à 20:25:09 SolarienKsander a écrit :
Y'a quoi dans tes commentaires ?

Bah j'explique pourquoi c'est faux, ou pourquoi on peut faire autrement.

T'as fait ça sans qu'elle te l'ait demandé ?

C'est quoi PR c'est un programme pour coder ?

Le 22 juin 2022 à 20:25:51 AutisteCouillu a écrit :

Le 22 juin 2022 à 20:25:09 SolarienKsander a écrit :
Y'a quoi dans tes commentaires ?

Bah j'explique pourquoi c'est faux, ou pourquoi on peut faire autrement.

Perso en tant que stagiaire j'apprécierais ce genre de chose, ça permet de progresser. Mais si c'est ta N+1...

Le 22 juin 2022 à 20:26:43 RoiCatholique a écrit :
T'as fait ça sans qu'elle te l'ait demandé ?

C'est quoi PR c'est un programme pour coder ?

Quand on code en équipe, avant de valider le changement (= Pull Request = PR), chaque membre de l'équipe est invité à regarder le code qui a été écrit pour déceler des éventuels bugs ou imprécisions.
Chacun laisse (ou non) des commentaires expliquant ce qui ne va pas, et après le codeur est chargé de mettre à jour son code ou répondre aux commentaires laissés.

Imagine t'écrit ton code qui marche plutôt bien, tu le publies, et t'as un connard qui toutes les 10 lignes laisse un commentaire pour dire que t'as pas codé de manière optimale...

Le 22 juin 2022 à 20:27:24 Dereliction a écrit :

Le 22 juin 2022 à 20:25:51 AutisteCouillu a écrit :

Le 22 juin 2022 à 20:25:09 SolarienKsander a écrit :
Y'a quoi dans tes commentaires ?

Bah j'explique pourquoi c'est faux, ou pourquoi on peut faire autrement.

Perso en tant que stagiaire j'apprécierais ce genre de chose, ça permet de progresser. Mais si c'est ta N+1...

J'imagine pas sa réaction demain matin en ouvrant sa PR et en voyant qu'il y a plus de commentaires que de code...

Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?

C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.

Le 22 juin 2022 à 20:33:18 :

Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?

C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.

Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression

Le 22 juin 2022 à 20:29:34 :

Le 22 juin 2022 à 20:26:43 RoiCatholique a écrit :
T'as fait ça sans qu'elle te l'ait demandé ?

C'est quoi PR c'est un programme pour coder ?

Quand on code en équipe, avant de valider le changement (= Pull Request = PR), chaque membre de l'équipe est invité à regarder le code qui a été écrit pour déceler des éventuels bugs ou imprécisions.
Chacun laisse (ou non) des commentaires expliquant ce qui ne va pas, et après le codeur est chargé de mettre à jour son code ou répondre aux commentaires laissés.

Imagine t'écrit ton code qui marche plutôt bien, tu le publies, et t'as un connard qui toutes les 10 lignes laisse un commentaire pour dire que t'as pas codé de manière optimale...

Si t'as vraiment raison sur l'optimisation du code t'as bien fait.

Même si t'es casse couille puisque tu te hisse au niveau de ta hiérarchie et que tu rajoute du travail, l'objectif de ce que vous produisez c'est que ça marche au mieux.

Ça peut être mal pris en mode mêle toi de ton cul ou à l'inverse tu peux être considéré comme un élément de grande valeur.

Je te comprends au fond moi sur un texte je ne pourrai pas m'empêcher de corriger les fautes et rendre plus élégantes les tournures de phrases foireuses.

Mais ce que je dis marche uniquement si t'es un bon gros matrixé par le code qui a un certain talent. Tu verras bien ce qu'elle a à en dire et si finalement c'est pas ta vision qui est mal optimisée pour des raisons que tu n'avais pas en tête.

Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :

Le 22 juin 2022 à 20:33:18 :

Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?

C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.

Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression

Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.

Tu peux t'essayer à l'asyncio si tu maîtrises pas.