FTKONF-070
Amikor a junior beosztottad panaszkodik a seniorok szakmai véleményére
Helyzet
„Hogyan kezeljem a juniort, aki panaszkodik, hogy a szenior telezúdítja kommentekkel a code review-it?” · „How to manage a junior developer who complains that a senior developer is throwing lots of comments on their code reviews?” · szerző: work_tech · CC BY-SA
Egy felsővezetőhöz jut el a panasz: a junior fejlesztő betartja a cég lefektetett kódolási sztenderdjeit, az egyik szenior mégis hosszú és nagyszámú kommentet zúdít a code review-ira, ami a juniort egyre jobban bosszantja. A kérdés nem az érintettek egyikéé, hanem a vezetőé: hogyan kezelje a kettejük közti súrlódást úgy, hogy a folyamatok elvileg rendben vannak? Az eredeti kérdés rövid és kifejtetlen; a tanulságot a közösség válaszai adják: hol a helye egy ilyen konfliktus kezelésének a szervezetben, és mitől lesz egy review-kultúra tanító, nem pedig őrlő.
A forrásposzt teljes szövege (magyarul és angolul)
Az eredeti Stack Exchange-kérdés hű magyar fordítása · szerző: work_tech · The Workplace, q185185 · CC BY-SA.
Hogyan kezeljünk egy junior fejlesztőt, aki arra panaszkodik, hogy egy szenior fejlesztő rengeteg kommentet szór a code review-ira?
Hogyan kezelnéd ezt felsővezetőként? Úgy, hogy már adottak a kódolási sztenderdek, folyamatok stb.
A kódolási sztenderdek már le vannak fektetve, és a junior mérnök követi is ezeket a sztenderdeket, a szenior mérnök mégis meglehetősen hosszú és nagy számú kommentet zúdít a junior mérnök kódjára. Ez bosszantja a junior mérnököt.
Original Stack Exchange question · author: work_tech · The Workplace, q185185 · CC BY-SA.
How to manage a junior developer who complains that a senior developer is throwing lots of comments on their code reviews? (Updated with more info)
How would you handle this as a senior manager? Given that there are already coding standards, process etc.
There are coding standards already set and the junior engineer is following the standards, yet the senior engineer is throwing pretty long and a number of comments on the junior engineer's code. This is annoying the junior engineer.
A közösség legjobbnak ítélt válasza (6 szavazat)
Szerző: Ertai87 · CC BY-SA
„This is not your problem. You are a senior manager. … These issues should be handled way below your level. … [From the original answer:] Telling someone to do something a particular way isn't helpful … Explaining to them why they shouldn't make that mistake is more helpful … If you make a comment on a PR but are unable to back up that comment with rationale or logic, then perhaps that »coding standard« is not as important as you think it is."
Lényeg (magyarul): Felsővezetőként ez nem a te asztalod; ha mégis hozzád jutott, a közbülső vezetői szint nem kezeli a konfliktust: ott kérj számon, oda delegáld vissza. A közvetlen vezetőnek szóló (eredeti) válasz gyakorlati receptje: a szenior engedje át a kódot, ha az eléri a saját mércéje ~80%-át: a maradék 20%-on vitázni nem éri meg; minden kommenthez járjon indoklás (a „miért" tanít, a puszta „csináld így" csak súrlódást szül; ami nem indokolható, az talán nem is fontos szabály); ha viszont a junior vesz minden kommentet személyes támadásnak, vele kell perspektíva-beszélgetést folytatni („ő 6 éve csinálja, te 6 hónapja, tanulhatsz tőle, ha hagyod"). Kiegészítő válasz (DarkCygnus, 0): először a meglévő sztenderdekre és folyamatokra mutass rá, és magyarázd el a review-kommentek értelmét csapatmunkában.
Valós forrás: „How to manage a junior developer who complains that a senior developer is throwing lots of comments on their code reviews?” · szerző: work_tech · Stack Exchange / The Workplace (q185185) · CC BY-SA · magyar fordítás, anonimizálva. Az eredeti poszt →