[dev] les code reviews on en parle ?

Le 02 octobre 2023 à 23:17:55 :
"Ça fonctionne nickel, par-contre je vois que tu ne suis pas bien les conventions pour le nommage des variables... recommence"

Oui, quand ton code est inspiré des modules du framework avec les mêmes conventions de nommage.

"Ah mais dans ce cas faut nommer ta config comme ça" => le framework va dans mon sens
"Ah oui mais c'est pas clair" => lit plus le code source du framework fdp au lieu de penser que tu détiens la vérité tout seul dans ton coin

Le 02 octobre 2023 à 23:19:42 :

Le 02 octobre 2023 à 23:17:55 :
"Ça fonctionne nickel, par-contre je vois que tu ne suis pas bien les conventions pour le nommage des variables... recommence"

Oui, quand ton code est inspiré des modules du framework avec les mêmes conventions de nommage.

"Ah mais dans ce cas faut nommer ta config comme ça" => le framework va dans mon sens
"Ah oui mais c'est pas clair" => lit plus le code source du framework fdp au lieu de penser que tu détiens la vérité tout seul dans ton coin

Ok j'ai l'impression que c'est + un mec inexpérimenté qui surestime ses capacités. C'est généralement ceux qui sont le plus réticents à l'apprentissage, trop d'égo. Il va sûrement finir manager.

Le 02 octobre 2023 à 23:24:05 :
Qui m'explique en tant que non geekos svp

- Naming conventions "J'utilise les conv du framework et de la communauté open source". "J'aime pas"
- convention vs configuration
"En général on fait le moins de code possible et on s'appuie sur des conventions cf python", "je sais pas, c'est pas explicite "
- Pas assez explicite : "oui mais c'est intégré au lifecycle du framework". "Je connais pas et flemme de lire la doc.. Request change"

Le 02 octobre 2023 à 23:25:32 :

Le 02 octobre 2023 à 23:19:42 :

Le 02 octobre 2023 à 23:17:55 :
"Ça fonctionne nickel, par-contre je vois que tu ne suis pas bien les conventions pour le nommage des variables... recommence"

Oui, quand ton code est inspiré des modules du framework avec les mêmes conventions de nommage.

"Ah mais dans ce cas faut nommer ta config comme ça" => le framework va dans mon sens
"Ah oui mais c'est pas clair" => lit plus le code source du framework fdp au lieu de penser que tu détiens la vérité tout seul dans ton coin

Ok j'ai l'impression que c'est + un mec inexpérimenté qui surestime ses capacités. C'est généralement ceux qui sont le plus réticents à l'apprentissage, trop d'égo. Il va sûrement finir manager.

50+ dev "senior", mais bon pas forcément meilleur que des dev de 30+
Déjà si tu lis ni la doc ni le code des lib, c'est mal parti.

Le 02 octobre 2023 à 23:29:10 :

Le 02 octobre 2023 à 23:25:32 :

Le 02 octobre 2023 à 23:19:42 :

Le 02 octobre 2023 à 23:17:55 :
"Ça fonctionne nickel, par-contre je vois que tu ne suis pas bien les conventions pour le nommage des variables... recommence"

Oui, quand ton code est inspiré des modules du framework avec les mêmes conventions de nommage.

"Ah mais dans ce cas faut nommer ta config comme ça" => le framework va dans mon sens
"Ah oui mais c'est pas clair" => lit plus le code source du framework fdp au lieu de penser que tu détiens la vérité tout seul dans ton coin

Ok j'ai l'impression que c'est + un mec inexpérimenté qui surestime ses capacités. C'est généralement ceux qui sont le plus réticents à l'apprentissage, trop d'égo. Il va sûrement finir manager.

50+ dev "senior", mais bon pas forcément meilleur que des dev de 30+
Déjà si tu lis ni la doc ni le code des lib, c'est mal parti.

Voilà le problème c'est que ton dev senior c'est un dinosaure. Discute avec lui, tu dis que vous êtes assez intelligents pour pouvoir bosser ensemble s'en vous envoyer des pics mais en attendant toi tu fais un topic sur lui sur JVC.

Cependant je compatis, je travaille avec mon ancien maître de stage d'il y a 10 ans depuis quelques jours, j'ai fais un tour dans d'autres sociétés entre-temps où j'ai appris un paquet de choses et lui en 10 ans il n'a pas évolué et il code toujours avec des outils qui datent de Mathusalem.

Il m'a prit de haut quand je lui ai conseillé de se renseigner sur de nouvelles technos/framework ou que je lui ai dis que certaines fonctions étaient dépréciées dans du code récent.

Le 02 octobre 2023 à 23:39:38 :

Le 02 octobre 2023 à 23:29:10 :

Le 02 octobre 2023 à 23:25:32 :

Le 02 octobre 2023 à 23:19:42 :

Le 02 octobre 2023 à 23:17:55 :
"Ça fonctionne nickel, par-contre je vois que tu ne suis pas bien les conventions pour le nommage des variables... recommence"

Oui, quand ton code est inspiré des modules du framework avec les mêmes conventions de nommage.

"Ah mais dans ce cas faut nommer ta config comme ça" => le framework va dans mon sens
"Ah oui mais c'est pas clair" => lit plus le code source du framework fdp au lieu de penser que tu détiens la vérité tout seul dans ton coin

Ok j'ai l'impression que c'est + un mec inexpérimenté qui surestime ses capacités. C'est généralement ceux qui sont le plus réticents à l'apprentissage, trop d'égo. Il va sûrement finir manager.

50+ dev "senior", mais bon pas forcément meilleur que des dev de 30+
Déjà si tu lis ni la doc ni le code des lib, c'est mal parti.

Voilà le problème c'est que ton dev senior c'est un dinosaure. Discute avec lui, tu dis que vous êtes assez intelligents pour pouvoir bosser ensemble s'en vous envoyer des pics mais en attendant toi tu fais un topic sur lui sur JVC.

Cependant je compatis, je travaille avec mon ancien maître de stage d'il y a 10 ans depuis quelques jours, j'ai fais un tour dans d'autres sociétés entre-temps où j'ai appris un paquet de choses et lui en 10 ans il n'a pas évolué et il code toujours avec des outils qui datent de Mathusalem.

Il m'a prit de haut quand je lui ai conseillé de se renseigner sur de nouvelles technos/framework ou que je lui ai dis que certaines fonctions étaient dépréciées dans du code récent.

Je suis dev senior++ et j'ai mis beaucoup de choses en place dans la boite (mon tech lead et manager tech me font confiance).

Pour être honnête, j'étais parti de la mission (dev freelance), je suis revenu 1 an après et j'ai vu la merde. J'ai tout remis en ordre car c'était n'importe quoi. Et mes tech lead + manager m'ont dit qu'heureusement que je suis là. Le TL était pas content mais n'avait pas les arguments tech pour prendre les décisions.. je suis revenu j'ai tout remis à plat en dégommant la merde qui avait été faite.

Il est pas mauvais, il a de bonnes idées en terme d'archi, surtout côté archi applicative. Mais le code c'est pas ça, sans parler de productivité (je fais 30+ SP quand il en fait 8...).

Mais bon on est en "démocratie", malgré le fait que le niveau soit pas le même, les codes reviews peuvent être bloquées pour des bêtises liées à l'incompétence d'un dev car chacun a le même pouvoir de "décision"...

[22:56:28] <Belojir>
Si tu as autant mal vécu le fait qu'un junior te fasse une remarque au point d'en faire un topic alors travaille sur ta confiance en toi

T'as pas réussi à remettre cette personne à sa place?

Le malaise de ces posts.

Le 03 octobre 2023 à 00:12:46 :
Il s'agirait d'être meilleur l'op

le pb n'est pas lié à mon niveau tech mais à celui des autres

Le 03 octobre 2023 à 00:20:51 :
Ça m'est jamais arrivé, le but des reviews c'est d'avoir quelqu'un d'autant/de plus expérimenté que toi pour commenté ton code et ta manière de faire dans le but de l'améliorer, c'est quoi ta boite de trisos ?

Une grosse boite. Avec plein de presta "senior"... sur le papier

Up, d'autres kheys confrontés à ce même problème ?
On évalue des complexités pour les SP des tâches, on sait très bien que quand un dev va faire ça en 2 jours l'autre prendra 5 jours

Mais chacun a son mot à dire dans les chiffrages
idem pour les code reviews.