Developpeur JUNIOR : je reçois JAMAIS de MERGE REQUEST en REVIEW

Le 15 avril 2024 à 19:43:41 :

Le 15 avril 2024 à 19:41:11 :

Le 15 avril 2024 à 19:39:44 :

Le 15 avril 2024 à 19:35:42 :
Comment ça tu ne reçois pas de MR ?
C'est pas toi qui reçoit les MR c'est le lead dev en général, et toi tu lui envoies des MR. Après je sais pas comment fonctionne ta boite mais j'imagine comme 90% des boites agile mes couilles avec un système de sprint et de tickets, donc logiquement t'as une phase de dev, puis tout est mergé sur la branche de dev par le lead et ensuite test / release.

c'est grossièrement ça

Y'a une feature branch, on fait une branche par sous tâche, une fois terminé on merge sur la feature branch, on dev, et la Q.A. fait son boulot une fois mergé sur cette feature branche

Sauf que je galère a trouvé des reviewers et on m'envoie jamais de reviews

Mais je comprends pas y a pas UN lead (ou product owner ou autre terme bullshit pour signifier au desuss du dev) dans ton équipe qui se charge de review / merge ?
Et si tu fais un dev et que personne le review il se passe quoi ? Ta PR reste stagnante ?

Il y a en a un, il est officiellement responsable de mes reviews, mais en pratique, il n'a pas review une seule fois mes tâches depuis plus d'un an maintenant

Je peux merger comme je veux, y'a aucune contrainte technique à merger sans reviews, c'est juste que si tu provoques une connerie avec une M.R. sans review, on va te le reprocher

Malgré ça je fais tout ce que je peux pour faire reviews mes tâches par contre.

Mais c'est toujours des approbations de juniors

Ok la gueule de l'organisation de ta boîte quoi (moi j'ai très rarement eut de review du code par contre c'était systématiquement mergé par la même personne qui assume conjointement si ça plante quoi).
Après le dev reste ta responsabilité et celle de ton équipe si y a des phases de test, le reviewer peut pas être sûr à 100% du résultat obtenu / attendu mais juste il est censé valider que grosso modo ça a l'air de coller quoi et / ou te faire remarquer que tu fais du doublon de code, ou des choses pas claires etc.
C'est une startup ou une vraie PME qui a de l'ancienneté ?

et puis c'est pas comme si tu faisais un metier utile, tu bosses pas chez OpenAI a ce que je sache

ça a l'air tellement chiant le métier de dev en entreprise

alors que quand tu es auto-entrepreneur et que tu bosses sur tes propres projets, là tu te fais plaisir

Le 15 avril 2024 à 19:48:08 :
ça a l'air tellement chiant le métier de dev en entreprise

alors que quand tu es auto-entrepreneur et que tu bosses sur tes propres projets, là tu te fais plaisir

Ça dépend, quand t'es dans une bonne boîte c'est intéressant car tu apprends énormément chaque jour

Le 15 avril 2024 à 19:46:25 :

Le 15 avril 2024 à 19:43:41 :

Le 15 avril 2024 à 19:41:11 :

Le 15 avril 2024 à 19:39:44 :

Le 15 avril 2024 à 19:35:42 :
Comment ça tu ne reçois pas de MR ?
C'est pas toi qui reçoit les MR c'est le lead dev en général, et toi tu lui envoies des MR. Après je sais pas comment fonctionne ta boite mais j'imagine comme 90% des boites agile mes couilles avec un système de sprint et de tickets, donc logiquement t'as une phase de dev, puis tout est mergé sur la branche de dev par le lead et ensuite test / release.

c'est grossièrement ça

Y'a une feature branch, on fait une branche par sous tâche, une fois terminé on merge sur la feature branch, on dev, et la Q.A. fait son boulot une fois mergé sur cette feature branche

Sauf que je galère a trouvé des reviewers et on m'envoie jamais de reviews

Mais je comprends pas y a pas UN lead (ou product owner ou autre terme bullshit pour signifier au desuss du dev) dans ton équipe qui se charge de review / merge ?
Et si tu fais un dev et que personne le review il se passe quoi ? Ta PR reste stagnante ?

Il y a en a un, il est officiellement responsable de mes reviews, mais en pratique, il n'a pas review une seule fois mes tâches depuis plus d'un an maintenant

Je peux merger comme je veux, y'a aucune contrainte technique à merger sans reviews, c'est juste que si tu provoques une connerie avec une M.R. sans review, on va te le reprocher

Malgré ça je fais tout ce que je peux pour faire reviews mes tâches par contre.

Mais c'est toujours des approbations de juniors

Ok la gueule de l'organisation de ta boîte quoi (moi j'ai très rarement eut de review du code par contre c'était systématiquement mergé par la même personne qui assume conjointement si ça plante quoi).
Après le dev reste ta responsabilité et celle de ton équipe si y a des phases de test, le reviewer peut pas être sûr à 100% du résultat obtenu / attendu mais juste il est censé valider que grosso modo ça a l'air de coller quoi et / ou te faire remarquer que tu fais du doublon de code, ou des choses pas claires etc.
C'est une startup ou une vraie PME qui a de l'ancienneté ?

C'est une boîte de trading qui a plus de 30 ans :ahi:

On peut pas lancer notre code, donc impossible de confirmer si ce qu'on a fait va fonctionner, donc les reviewers sont obligés de vraiment analyser la solution de fond en comble

C'est aussi la raison pour laquelle je fais tout ce que je peux pour faire des tests unitaires d'ailleurs, puisque ça me permet au moins de confirmer que les parties que je tests unitairements fonctionnent correctement

Sinon vous codez des trucs utiles des fois ou c'est que ça votre boulot ?

Le 15 avril 2024 à 19:52:35 :
Sinon vous codez des trucs utiles des fois ou c'est que ça votre boulot ?

Ce sur quoi je travail actuellement :

Les Indications of Interests

Quand tu envoies un ordre au marché pour faire une commande sur un instrument, il faut envoyer automatiquement un ensemble de notifications aux différents traders de plusieurs clients, et aux clients eux même via le protocol FIX

Ces notifications seront un faux jumeaux de cet ordre en fonction d'un ensemble de conditions pour le transformer et avertir un ensemble de client que tel quantité est disponible à l'achat/vente

J'ai littéralement tout fait from scratch sur ce workflow actuellement

Le 15 avril 2024 à 19:57:02 27AnsEtPuceau a écrit :

Le 15 avril 2024 à 19:52:35 :
Sinon vous codez des trucs utiles des fois ou c'est que ça votre boulot ?

Ce sur quoi je travail actuellement :

Les Indications of Interests

Quand tu envoies un ordre au marché pour faire une commande sur un instrument, il faut envoyer automatiquement un ensemble de notifications aux différents traders de plusieurs clients, et aux clients eux même via le protocol FIX

Ces notifications seront un faux jumeaux de cet ordre en fonction d'un ensemble de conditions pour le transformer et avertir un ensemble de client que tel quantité est disponible à l'achat/vente

J'ai littéralement tout fait from scratch sur ce workflow actuellement

ca a l'air nul a chier, dev en banque c'est le pire milieu. Trouve toi une boite avec un produit interessant

Le 15 avril 2024 à 19:59:56 :

Le 15 avril 2024 à 19:57:02 27AnsEtPuceau a écrit :

Le 15 avril 2024 à 19:52:35 :
Sinon vous codez des trucs utiles des fois ou c'est que ça votre boulot ?

Ce sur quoi je travail actuellement :

Les Indications of Interests

Quand tu envoies un ordre au marché pour faire une commande sur un instrument, il faut envoyer automatiquement un ensemble de notifications aux différents traders de plusieurs clients, et aux clients eux même via le protocol FIX

Ces notifications seront un faux jumeaux de cet ordre en fonction d'un ensemble de conditions pour le transformer et avertir un ensemble de client que tel quantité est disponible à l'achat/vente

J'ai littéralement tout fait from scratch sur ce workflow actuellement

ca a l'air nul a chier, dev en banque c'est le pire milieu. Trouve toi une boite avec un produit interessant

C'est pas une banque

Moi j'aime bien ce workflow

J'ai rien compris, tu crée des merges request. Personne les review et tu les merge sans review ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Où bien tu veux faire des review de code a tes collegues ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

Le 15 avril 2024 à 20:04:27 :
J'ai rien compris, tu crée des merges request. Personne les review et tu les merge sans review ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Où bien tu veux faire des review de code a tes collegues ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

je merge mes M.R. avec seulement des approvals de juniors, les seniors ne me reviewent pas

J'aimerais faire des reviews pour d'autres mais je n'en reçois pas.

Le 15 avril 2024 à 20:07:16 :

Le 15 avril 2024 à 20:04:27 :
J'ai rien compris, tu crée des merges request. Personne les review et tu les merge sans review ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Où bien tu veux faire des review de code a tes collegues ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

je merge mes M.R. avec seulement des approvals de juniors, les seniors ne me reviewent pas

J'aimerais faire des reviews pour d'autres mais je n'en reçois pas.

Et tu as pas la visibilité sur les M.R de tes collegues ? Car moi je peux voir les M.R de l'ensemble du projets en tant que dev (bon j'ai les droits admin car j'ai config plein de trucs).
Sinon tu peux très bien voir les commits de tes collegues et en discuté avec eux.

Je sais que sur ma 1ere boite j'avais mis un trucs qui me notifié des commits/push des collegues pour ensuite faire des revues de code en livehttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Comme ca s'il fallait je faisait une revue de code via un retroprojecteur pour qu'on discute des axes à améliorer

Le 15 avril 2024 à 20:09:52 :

Le 15 avril 2024 à 20:07:16 :

Le 15 avril 2024 à 20:04:27 :
J'ai rien compris, tu crée des merges request. Personne les review et tu les merge sans review ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Où bien tu veux faire des review de code a tes collegues ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

je merge mes M.R. avec seulement des approvals de juniors, les seniors ne me reviewent pas

J'aimerais faire des reviews pour d'autres mais je n'en reçois pas.

Et tu as pas la visibilité sur les M.R de tes collegues ? Car moi je peux voir les M.R de l'ensemble du projets en tant que dev (bon j'ai les droits admin car j'ai config plein de trucs).
Sinon tu peux très bien voir les commits de tes collegues et en discuté avec eux.

Je sais que sur ma 1ere boite j'avais mis un trucs qui me notifié des commits/push des collegues pour ensuite faire des revues de code en livehttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Comme ca s'il fallait je faisait une revue de code via un retroprojecteur pour qu'on discute des axes à améliorer

je le fais tout le temps

Je pensais être un weirdo à checker tout le temps le travail de mes collègues

Mais par contre je discute pas avec eux, si c'est pas un workflow sur lequel je risque de travaillais généralement j'essaye vaguement de comprendre ce qu'il se passe et je regarde si y'a des points techniques intéressants

C'est une très bonne idée les codes reviews en projecteur

Le 15 avril 2024 à 20:13:47 :

Le 15 avril 2024 à 20:09:52 :

Le 15 avril 2024 à 20:07:16 :

Le 15 avril 2024 à 20:04:27 :
J'ai rien compris, tu crée des merges request. Personne les review et tu les merge sans review ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Où bien tu veux faire des review de code a tes collegues ?https://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

je merge mes M.R. avec seulement des approvals de juniors, les seniors ne me reviewent pas

J'aimerais faire des reviews pour d'autres mais je n'en reçois pas.

Et tu as pas la visibilité sur les M.R de tes collegues ? Car moi je peux voir les M.R de l'ensemble du projets en tant que dev (bon j'ai les droits admin car j'ai config plein de trucs).
Sinon tu peux très bien voir les commits de tes collegues et en discuté avec eux.

Je sais que sur ma 1ere boite j'avais mis un trucs qui me notifié des commits/push des collegues pour ensuite faire des revues de code en livehttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Comme ca s'il fallait je faisait une revue de code via un retroprojecteur pour qu'on discute des axes à améliorer

je le fais tout le temps

Je pensais être un weirdo à checker tout le temps le travail de mes collègues

Mais par contre je discute pas avec eux, si c'est pas un workflow sur lequel je risque de travaillais généralement j'essaye vaguement de comprendre ce qu'il se passe et je regarde si y'a des points techniques intéressants

C'est une très bonne idée les codes reviews en projecteur

Perso je le fais plus trop maintenant, je suis dans une start up de 3 personnes avec bonne ambiance. Même si le dev front est clairement nul et je dois h24 l'aidé alors que je suis pas dev front a la base. Mais bon je suis multiple casquette mtn, je fais du dev / dev ops etchttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

Tu dois aider le dev front sur quel point ?

L'interaction avec le BE ou alors sur des point purement FE ?

Si c'est le deuxième point y'a un problème

Le 15 avril 2024 à 20:28:17 :
Tu dois aider le dev front sur quel point ?

L'interaction avec le BE ou alors sur des point purement FE ?

Si c'est le deuxième point y'a un problème

Tout, il a aucune logique de dev. Il refait des fonctions en boucle plutot que de l'avoir a un seul endroit. Au pif, il avait recodé une fonction "goTo" sur chacune de ses classeshttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png
Parfois je dois prendre des sujets front car il est trop en retard et il a du mal. Du coup le patron me demande de le faire moi car je fais vite et bienhttps://image.noelshack.com/fichiers/2022/37/1/1663014384-ahi-pince-mais.png

Insupportable les boites avec du code de basse qualité

perso je vois pas le soucis, personne t'emmerde.

Le 15 avril 2024 à 19:35:18 :

Le 15 avril 2024 à 19:33:54 :

Le 15 avril 2024 à 19:33:21 :

Le 15 avril 2024 à 19:32:40 :
On approche des 2 ans, il serait temps de prospecter pour une augmentationhttps://image.noelshack.com/fichiers/2021/23/5/1623422583-ahiaffaire.png

j'ai déjà démissionnés, je pars le 27 juin

Je supportais pas cette impression d'inutilité

T'as un truc derrière j'espèrehttps://image.noelshack.com/fichiers/2021/23/5/1623422583-ahiaffaire.png

non

Ahi, pas le bon moment pour partir sans avoir rien derrièrehttps://image.noelshack.com/fichiers/2021/23/5/1623422583-ahiaffaire.png