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

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Le 22 juin 2022 à 21:29:18 Bychomeuring a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Change de boîte kheyou, tu mérites mieux que ça, tu es sous-valorisé.

Le 22 juin 2022 à 21:32:41 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:29:18 Bychomeuring a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Change de boîte kheyou, tu mérites mieux que ça, tu es sous-valorisé.

En parlant de ça , j'ai reçu un mail linkedin d'une boite concurrente assez sérieuse et pas de tutoiement à la con.
Tu m'a motivé à leu répondre

Le 22 juin 2022 à 21:33:58 Bychomeuring a écrit :

Le 22 juin 2022 à 21:32:41 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:29:18 Bychomeuring a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Change de boîte kheyou, tu mérites mieux que ça, tu es sous-valorisé.

En parlant de ça , j'ai reçu un mail linkedin d'une boite concurrente assez sérieuse et pas de tutoiement à la con.
Tu m'a motivé à leu répondre

Reste pas coincé au même endroit par flemme du changement, c'est le pire. Bon chance.

Le 22 juin 2022 à 21:36:26 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:33:58 Bychomeuring a écrit :

Le 22 juin 2022 à 21:32:41 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:29:18 Bychomeuring a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Change de boîte kheyou, tu mérites mieux que ça, tu es sous-valorisé.

En parlant de ça , j'ai reçu un mail linkedin d'une boite concurrente assez sérieuse et pas de tutoiement à la con.
Tu m'a motivé à leu répondre

Reste pas coincé au même endroit par flemme du changement, c'est le pire. Bon chance.

C'est vrai que j'ai tendance à rester par confort
merci et bonne chance à toi aussi

Le 22 juin 2022 à 21:37:52 Bychomeuring a écrit :

Le 22 juin 2022 à 21:36:26 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:33:58 Bychomeuring a écrit :

Le 22 juin 2022 à 21:32:41 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:29:18 Bychomeuring a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

Nan mais quand c'est la dixième fois que leur dit, tu laisses passer
Du coup, je me rends co-responsable d'un code pas fou

Change de boîte kheyou, tu mérites mieux que ça, tu es sous-valorisé.

En parlant de ça , j'ai reçu un mail linkedin d'une boite concurrente assez sérieuse et pas de tutoiement à la con.
Tu m'a motivé à leu répondre

Reste pas coincé au même endroit par flemme du changement, c'est le pire. Bon chance.

C'est vrai que j'ai tendance à rester par confort
merci et bonne chance à toi aussi

Merci.

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

En quoi c'est une mauvaise chose les conditions ternaires ?

Le 22 juin 2022 à 21:41:20 [TourteAuFoutre a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

En quoi c'est une mauvaise chose les conditions ternaires ?

Regarde le code : le mec a fait un if/else au lieu d'utiliser le ternaire justement.
C'était à ça que je faisais référence, pas à l'opérateur ternaire. C'était ambigu désolé.

Le 22 juin 2022 à 21:42:29 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:41:20 [TourteAuFoutre a écrit :

Le 22 juin 2022 à 21:27:11 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:20:50 Bychomeuring a écrit :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Oui pareil je me retiens. Si le code est valide, en général je ferme les yeux sauf si y'a moyen de beaucoup simplifier.

Là ton ternaire j'aurais jamais laissé passer... Sûrement qu'ils auraient appris quelque chose en plus. Pour mieux l'oublier une semaine plus tard...

En quoi c'est une mauvaise chose les conditions ternaires ?

Regarde le code : le mec a fait un if/else au lieu d'utiliser le ternaire justement.
C'était à ça que je faisais référence, pas à l'opérateur ternaire. C'était ambigu désolé.

Ah ok

Je me rends compte que je suis pas prêt de progresser dans ma boîte avec nos méthodes de développement

Y'a pas de review par les collègues, quand mon supérieur corrige mon code il le change sans rien me dire, je sais simplement que mon code marche et ça part comme ça

Alors que là avec les conseils en or de l'auteur ce serait génial, j'apprendrai plein de choses

À mon ancien taf on avait un architecte qui faisait les revues de code, mais lui ne faisait que les commentaires chiants style l'indentation est pas bonne, il faut renommer la fonction, il manque des brackets, la variable est pas bien nommée ahi

Je fais pareil à mon tour sur mon job actuel, notamment sur les reviews du code des stagiaires

Le 22 juin 2022 à 21:20:50 :

Le 22 juin 2022 à 21:15:50 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:15:08 Bychomeuring a écrit :

Le 22 juin 2022 à 21:14:05 AutisteCouillu a écrit :

Le 22 juin 2022 à 21:12:11 Bychomeuring a écrit :
Je fais pas mal de commentaires aussi. J'ai bien précisé à plusieurs reprises à l'équipe que mes commentaires, c'est plus des questions/suggestions que la vérité absolue.
Mais je bosse avec des flemmards qui insèrent bêtement mes suggestions sans discuter et qui résolvent le thread sans commentaire

Aaaaaya. Pareil kheyou.

En plus souvent je me fais chier à donner des explications et à donner des liens vers des articles confortant ma suggestion, mais je sais même pas s'ils y prêtent attention...

Non ils s'en foutent, ils veulent juste boucler leur ticket sans se prendre la tête

Dans quelle merde on s'est foutu mon khey...

Alors que un des interets principaux de la revue de code, c'est l'échange et l'amélioration du code de l'équipe.
Y''a des fois où je laisse même passer des trucs cons car je sais qu'on me deteste

Genre le ternaire

if (string!=null) {
string2 = string;
} else {
string2 = "toto";
}
Alors que
string2 = string!=nul ? string : "toto";

Tu demandes trop déjà
Les golems n'utilisent pas de ternaire

Le 23 juin 2022 à 09:45:35 AutisteCouillu a écrit :
Bon ben elle faisait la gueule ce matin au daily meeting...