how we write/review code in big tech companies

LGTM = Looks good to me

😂

this video is fun to link to people in pullrequests

This makes me feel good. I feel so dumb at work 😂 (SE)

Not may case, when I push code, I have to prepare a "presentation" to explain what I am doing because he will leave a lot of comments

Lol. Wasnt paying attention before. Totally on the senior for approving a CL without asking why he deleted a unit test

Me reading his message: "I have no idea what that is short for."

😂

Was there a second part where the junior software dev gets a phone call because everything fails now?

I thought it was only In residency in medicine 😂lol
I Feel better thanks

Git commit -m "some changes"

man that all day?

Hahahaha! I like that

Exactly what happens in my company..

Wth is porndora

I'm an aircraft engineer and thank Fk I don't do this. People would die

doing it right now! Approving pull request without even understanding what is it about :D

This is the best thing I've seen in a while thank you thank you😍😍😍

Removing test case will lead to reduce code coverage metric.
Despite this is a bullshit metric, it is used in enterprise.

Yeah, eveyone just assuming everyone else knows what they are doing is exactly how to get stuff done. If it compiles, thats progress.

Nice

Hahahahah so true

Literally me on every community poll.

This is amazing!

no one knows what is he doing 😂😂😂😂😂😂😂😂

The difference between a senior and junior is the senior comments the failing tests out while the junior deletes them entirely.

I swear that’s me with a senior level of python
And when my Junior sending me this shit I don’t know what the hell is this 😂

You don't get a cubicle? You have to sit at a table?

Your videos are brilliants

he deleted the test cases

Great work

imagine if actual engineers did this shit

Looks great to me!

씨맥 폼미쳤다

Remove the test code then problem fixed 😅

Yesterday I was in a meeting and a more experienced back-end engineer was going on how it made more sense to do a certain thing X way, I was like “yeah, that totally makes sense! We should do it that way”. After I closed the meeting the first thing I said to myself was “I have no idea wtf he was talking about”. This is too real omg

More like nobody responds lmfao

Халтурщики есть везде.
Из-за них распадаются компании.

Haha, very much relatable

There's 4000 people who actually know how to program and everyone else is an imposter.
I'm one of the 4000.

Ahh yes , yet another job soon to be made redundant by A.I. ,I've always thought these bastards were overpaid anyway, good riddance easy mode.

Why both the Junior and Senior feel the urge to take the shortcut? Is it the pressure of those people(technical support, sre, customers, account managers, MBAers not having written a line of code)? Is it the scrum process that encourages or mandates to mark your points and submit that specific day? I can't believe that someone deliberately wants to do a bad/half job while getting paid.

scammer

Same as us in the construction field,.. we had no idea what we were doing, just do ..the job until done 😅

"Its a feature not a bug"

Usually the seniors only see problems in the naming, indentation, and redundancy.

But if there’s a runtime error or logic error. It slips by relatively easy 😂.

Remember to write your tests along with your implementations!

En todas partes del mundo pasa lo mismo jajajajaja
Saludos desde Chile !

>test fails
>deletes test case
>test no longer fails

The accuracy of this