FTKONF-025
Amikor a senior kolléga folyamatosan megkérdőjelez
Helyzet
„A szenior így vagy úgy, de meg akar szabadulni tőlem” · „Senior wants me gone, one way or another” · szerző: bluetailedfox · CC BY-SA
Egy kezdő, első igazi állásában lévő fejlesztő a cég legfiatalabbja. Egy két évtizedes tapasztalatú szenior (látszólag a pozícióját/territóriumát féltve) szisztematikusan aláássa: a „ne vitatkozz a kódbírálattal, mert rontja a hírneved" üzenettől kezdve a „do not review" jelölésű draftok kötözködő bírálatán át odáig, hogy aláírás nélkül húzza a review-t, amíg a határidő lecsúszik, így a fiatal „alulteljesítőnek" tűnjön. Más kollégák pozitív visszajelzést adnak ugyanarra a kódra, a szenior mégis „rossz kódnak" minősíti, indoklás nélkül. A fiatal nem tudja, mit rontott el.
A forrásposzt teljes szövege (magyarul és angolul)
Az eredeti Stack Exchange-kérdés hű magyar fordítása · szerző: bluetailedfox · The Workplace, q168687 · CC BY-SA.
Kérdés: Egy szenior így vagy úgy, de meg akar szabadulni tőlem
Már majdnem egy éve dolgozom az első igazi tech-állásomban (fullstack webfejlesztőként) egy helyi cégnél. Ezt megelőzően volt 2 nyári gyakornoki munkám és néhány személyes projektem. A jelenlegi cégnél többnyire React-tel, Node-dal és Javascripttel dolgozom (aminek a megtanulása eltartott egy ideig, mivel az utolsó 2 nyári gyakornoki munkám is Pythonra fókuszált), de a mai napon mind én, mind néhány másik kolléga bízik a kódolási képességemben (az elmúlt 2 hónapban elég pozitív értékeléseket kaptam, sőt, néhány bókot is). Megjegyzendő, hogy én vagyok a legfiatalabb a cégnél (mind kor, mind tapasztalat tekintetében), és a cég struktúrája elvileg lapos (flat). A probléma az, hogy van egy több mint 2 évtizedes tapasztalattal rendelkező szenior, aki úgy tűnik, nem kedvel engem. Amikor először interakcióba léptem vele, eléggé csendesnek tűnt, de általában elég segítőkész és kész volt magyarázni. Most, egy idő után, határozottan elmondhatom, hogy meg akar szabadulni tőlem. Azért mondom ezt, mert több vészjelzést is felvetett: Tartott egy prezentációt a kódbírálati (code review) gyakorlatokról, ahol azt mondta, hogy nem szabad vitatkozni a kódoddal kapcsolatos kéréseken, mert az „tönkreteszi a hírnevedet", még akkor is, ha egy másik szenior azt mondta, hogy a kódváltoztatások megvitatása a kódbírálat lényege. (Ez azután jött, hogy megkérdeztem, miért kellene megvalósítanom a kért változtatást. Őszintén nem értettem, mert nekem a kódom jónak tűnt, de azt hittem, kihagytam valamit.) Aktívan bírálja a „Ne bíráld" (Do not review) címkével ellátott draft PR-jaimat, és olyan dolgokra mutat rá, amelyek még nincsenek kész (kommentek, nem használt változók, nem optimalizált kód). Az egyik menedzser elmondta nekünk, hogy néhány szenior panaszkodott az újabb alkalmazottakra, és hogy ők teljesen haszontalanok a cég számára. Követtem a termékmenedzserünk által adott specifikációkat, ő pedig azt mondta, hogy ezek a specifikációk nem jók, és hogy rossz programozó vagyok, amiért követem őket. Olyan kódrészleteken kért változtatásokat, ahol egy másik kolléga pozitív visszajelzést adott nekem. Megjegyzendő, hogy ez a szenior bírálta a kódomat, semmi problémásat nem talált benne (de valamiért nem hagyta jóvá a PR-t), majd jött a másik kolléga, pozitív visszajelzést adott (tetszett neki a megvalósításom) egy bizonyos kódrészletre, majd egy-két órával később az a szenior azonnali változtatásokat kért azon a kódrészleten, amely pozitív visszajelzést kapott, azt mondva, hogy „rossz kód" (minden más magyarázat nélkül, még azután is, hogy megkérdeztem, miért rossz). Beteszi magát fő bírálónak, majd szándékosan nem bírálja a kódot, csak a PR-ban megvalósított feature határideje után. Ettől a nála kevésbé szenior kollégák félnek jóváhagyni a PR-t, mert rossz viszonyba kerülnének vele. Ráadásul a nagy PR-oknál csak darabokban bírálja a kódot, vagyis átnézi az egész kódot, és kér egy változtatást ma (amit azonnal, perceken belül megvalósítok). Aztán a következő napon kér egy változtatást, de a kezdeti változtatás előtt írt kód alapján (alapvetően nem ad részletes kódbírálatot először, hogy egyfajta limbóban tarthasson, amíg a határidőm le nem jár, hogy alulteljesítőnek lehessen látni). Egyszerűen figyelmen kívül hagyja az összes közvetlen üzenetemet és kérdésemet hozzá (mint például a bírálati kérések). A többi kolléga közül senki nem csinál ilyet. Mindannyian profik és elég segítőkészek, de ez a fickó valahogy utál engem, és azt akarja, hogy kirúgjanak. Soha nem vettem részt irodai pletykában, mivel a pandémia miatt alig tettem be a lábam az irodába néhányszor. Soha nem sértettem meg, és nem kérdőjeleztem meg ennek a szeniornak a kódolási képességét. Soha nem tettem megjegyzést a szenior külsejére és hozzáállására. Tényleg nem akarok most állást keresni, de már nem sokáig bírom így. Mit tegyek? Vegyem fel a kapcsolatot a feletteseimmel, és mutassam meg nekik a bizonyítékokat?
Original Stack Exchange question · author: bluetailedfox · The Workplace, q168687 · CC BY-SA.
Senior wants me gone, one way or another
I've been working my first real tech job (as a fullstack web developer) at a local company for almost a year now. Prior to that, I had 2 summer internships and some personal projects. At the current company, I work mostly with React, Node and Javascript (which took a while to learn, as my last 2 summer internships were both Python focused), but as of today both me and a couple of other colleagues are confident in my ability to code (I've been receiving quite positive reviews in the last 2 months, even as far as some compliments). To be noted, that I'm the youngest in the company (both in terms of age and experience) and the company structure is supposed to be flat. The problem is that there is a senior with more than 2 decades of experience who seems to not like me. When I first interacted with him, he seemed pretty quiet, but usually quite helpful and willing to explain. Now, after a while, I can definitely say that he wants me gone. I am saying this because he raised several red flags: He held a presentation about code review practices, where he said that you should not argue about the requests on your code, because it will "ruin your reputation", even if another senior said that discussing about code changes is the whole point of code review. (This came after I asked why should I implement the change he requested. I genuinely did not understand, because to me my code seemed good, but I thought I missed out something.) He actively reviews my draft PRs which have the "Do not review" tag and points out things that are not done yet (comments, unused variables, unoptimised code). One of the managers told us that some seniors complained about newer employees, and that they are completely useless to the company. I was following the specifications given by our product manager and he said that this specifications are not good, and that I am a bad programmer for following them. He requested changes on pieces of code where another colleague gave me a positive feedback. Note that this senior reviewed my code, found nothing problematic with it (but didn't approve the PR for some reason), the other colleague came, gave me a positive feedback (he liked my implementation) for a certain piece of code, then an hour or two later, that senior requested immediate changes on the piece of code that received positive feedback, saying it was "bad code" (with no other explanation, even after I asked him why was it bad). He puts himself as the main reviewer then purposely doesn't review the code, only after the deadline of the feature implemented on the PR. This makes other colleagues which don't have the same seniority as him to be afraid to approve the PR, because they would get on bad terms with him. Also, for large PRs, he just reviews the code in chunks, meaning he reviews all the code, and requests a change today (which I implement right away, in a matter of minutes). Then, the following day, he requests a change, but based on the code that I wrote before the initial change (he basically doesn't give a detailed code review the first time, so he can keep me in some sort of limbo until my deadline passes so I can be seen as underperforming). He flat out ignores all of my direct messages and questions to him (such as requests for review). None of the other colleagues do this. They are all professional and quite helpful, but this guy somehow hates me and wants me fired. I never took part in office gossip, since I barely set foot in the office a few times, because of the pandemic. I never insulted or questioned this senior's ability to code. I never made any comments about the senior looks and attitude. I really don't want to start job hunting now, but I cannot go on like this for much longer. What should I do? Contact my superiors and show them the evidence?
A közösség legjobbnak ítélt válasza
Hilmar (111) · CC BY-SA
Talk to your manager. … Keep detailed notes. Describe the behavior and how it impacts your productivity … Don't judge, just stick to the facts. Tell your manager you'd be happy to change your code in whatever way is required, but you can't because you're not getting feedback on what's wrong. … HR should ONLY be a matter of last resort.
Magyar összefoglaló: ne a szeniorral próbáld „megnyerni" a háborút. Dokumentálj tényszerűen, vidd a vezetőhöz a hatásra fókuszálva (nem ítélkezve), kérj konkrét szakmai visszajelzést, és kérd, hogy a vezető teremtsen teret (ne dolgozzatok egy darabig ugyanazon).
Valós forrás: „Senior wants me gone, one way or another” · szerző: bluetailedfox · Stack Exchange / The Workplace (q168687) · CC BY-SA · magyar fordítás, anonimizálva. Az eredeti poszt →