When You Ask the Intern to Review Your Code
Ойын-сауық
what happens when you ask the intern for a code review....
it's been a while since i last made a skit, i hope you'll enjoy this one
if you're reading this, comment: weather magician
ig: / niicholas.t
discord: / discord
#intern #coding #softwareengineer
sub count: 146 404
Lurking Sloth by Alexander Nakarada (CreatorChords) | creatorchords.com
Music promoted by www.free-stock-music.com
Creative Commons / Attribution 4.0 International (CC BY 4.0)
creativecommons.org/licenses/...
Пікірлер: 314
The real intern experience is spending a week working on a PR, sending it in with everything ironed out perfectly, having crafted the best PR of your entire life, then seeing it rot on the task list with a low priority number for months
@weak1ings
7 ай бұрын
And then never getting merged until it is inevitably closed!
@rewrose2838
7 ай бұрын
Bro!! I thought I was the only one. I had fixed a major issue in production, gave the PR and it got merged after two months lol. I had literally zero tasks for those months
@AnibalAlvarez
7 ай бұрын
The best python package of my life was never even reviewed
@augustofotino523
7 ай бұрын
This gave me PTSD
@tilmanrotationalinvariant2257
7 ай бұрын
Months? More like years! I have in all seriousness a couple of PR's from over a year(when I started) open. Lets see if they are gonna be reviewed next year...
Intern manager had me dying 💀
Intern tension: "I'm new and am expected to know these things. I can't ask or they will think I don't have a clue what I'm doing. But I also don't feel like I know what I'm doing." Senior tension: "This kid must be brilliant because they haven't asked questions. They must know the latest/greatest practices so I can't question that. I don't understand this and I'm expected to teach them..." Both: LGTM
@AnibalAlvarez
7 ай бұрын
There are some words in that hudge fact
@KristianTheDesigner
7 ай бұрын
As a frontend dev who started just 6 months ago i need to say that this comment made me calm 😁 Thanks
@xgcwrought3346
7 ай бұрын
Omg is this what I get to look forward to?
@liloruf2838
6 ай бұрын
True! The best is when seniors just explain everything without me having to ask😂😂😭
That feeling of "He wants me to review *his* code?" is truly universal for software devs
Actually worked with an intern who manually changed his job title on the internal portal to 'senior software engineer intern' as it was his 4th internship at the company - made me giggle.
@matteosposato9448
6 ай бұрын
Fun and brilliant! But maybe also a way to say that rather than having a fourth internship one should be hired or at least get a consultancy? Don't know about the specific situation so maybe you'll tell more, but fourth internship (and at the same company!) sounds out of this world to me
@hunter-tm2kl
2 ай бұрын
@@matteosposato9448 i had a friend who did 4 separate terms at a big fintech company and he worked on a different team each time
@HeatFort
2 ай бұрын
@@matteosposato9448probably still in school so they were waiting to hire him
@marcialabrahantes3369
Ай бұрын
@@matteosposato9448I've seen that for students that are going into Master's program
switching variable colour to color has made it 20% efficient saving millions of dollars in cloud cost. keep up the great work !
@AryanGNU
7 ай бұрын
Lmao chuckled so hard 😂😂😂
@triplezgames3882
7 ай бұрын
Actually, "colour" is 20% less efficient than "color", but "color" is 16,67% more effective than "colour"
@magicmulder
7 ай бұрын
You can actually put 10% of the code in the filename. Huge space saver.
@hnazmul
6 ай бұрын
😂😂😂😂
Ohhhh my god. Sooo many Easter eggs in this video. This had to take so much time and also worth every second!
Not only did I review my mentor’s PR, I REJECTED it
The real intern experience is spending a week working on a PR, sending it out for review, then finding out the requirements were all wrong because the senior engineer that gave you direction was not on the same page as the other senior engineers.
@coherentpanda7115
7 ай бұрын
That's an every software developer experience, including for seniors.
@NevosLP
6 ай бұрын
Has happened so many times to me...
@marc-andrefortin
4 ай бұрын
Classic
This has no business being this good!! Insane how much work put into this, truly appreciate you.
@nicholast
7 ай бұрын
glad you enjoyed it
@adityasuryawanshi3263
7 ай бұрын
@@nicholastwhat is the name of that utility at the start of the video where you are managing all the tasks, looks cool?
@marjo9952
7 ай бұрын
@@adityasuryawanshi3263 "Trello" I'm assuming
after starting my first internship i can finally relate to all the intern memes and confirm they are 100% accurate
@ConfusedWatermelon
7 ай бұрын
congrats on internship!
@antekliyue9874
7 ай бұрын
so are you intern manager yet?
0:43 you forgot the holy title of INTERN CEO
@nicholast
7 ай бұрын
chief intern officer
@d_sanu
7 ай бұрын
That's a CIO... 😂 not CEO
@armtdawg99
3 ай бұрын
Chief Executive Intern
I recently got put on a new project at work and this is way too relatable / anxiety inducing! Well done, sir!
I'm a junior dev. My team recently had a new senior dev join. Senior dev clones a repo and starts making some changes and asks me, why doesn't their code work. I tell them to please push their code to another branch so I can review. One week later, sends me a zip file containing the whole project repo (doesn't know how to use Git Hub). First thing I see are a bunch of nested loops, 150 lines of if-else statements, a bunch of poorly named variables, and no comments. WTF. I'm dead.
@Haise-san
6 ай бұрын
That doesn't sound like a "senior" dev, but what do we juniors know lmao, maybe he was into something
@jakobullmann7586
6 ай бұрын
Been there, that’s real life. Maybe not at FAANG, but in smaller companies or teams this can definitely happen. Promotions based on seniority, not skill level… My advice: Stay away from those companies/teams. Work with skilled people and for companies that value skill.
@siliconhawk9293
3 ай бұрын
@@Haise-sanhe probably was on to something he didn't become a senior engineer just like that. right guys ?
@MrCmon113
3 ай бұрын
At least he zipped it. He could have sent a rider into your town to read the code out loud in the market square.
@angelg3642
2 ай бұрын
HOW HOW HOW HOW HOW DID THAT HAPPEN ????? My last interview I straight up outperformed 90% of the candidates and that was still not good enough. WHAT THE FUCK ????
I will definetely add the following point to my resume "Increased corporate profits by 27% by enhancing user experience by shifting critical ui component to utilize warm shades" paraphrase: I made a button red
@angelg3642
2 ай бұрын
Naaah you legit gotta do that 😂😂😂😂😂😂. The smarter it sounds the more impressed the HR's are. It's legit comedy sometimes
This made my day, great content!
love it man! Keep dropping.
video editing attention to detail goes craaazy !!
I showed your video to the Senior Intern on our team and he confirmed that this is how it goes.
Congrats on the promotion to Double Super Intern: First Class!
Welcome back man!!
Great content, was watching your vids while prepping for the interviews and they were great at boosting my morale. Now after getting a return offer this week, this video seems like a cherry on top.
IVE MISSED THESE VIDEOS
ayeee he is back with the quality content
loved the video. keep dropping more 👍
@nicholast
7 ай бұрын
MORE SKITS COMING
Another banger 🔥🔥🔥
this was hilarious 😂
Bro I relate to this so hard 😂
Good ol intern days they had me twiddle from thumbs for 4 weeks then gave me some random code story development with one sentence in rally describing what to do and then being busy for the whole week only to get the jr offshore developer to yell at me for what I was doing 😅. The managers would then not find any other work until it was two weeks until I was supposed to leave and act like they loved me.☠️
Was waiting for your videos :)
If CI passes, it's totally fine. I just started at a new company and haven't really learned some of the systems yet, so I just depend on compiler errors and tests to tell me if I'm breaking things. Just like an intern.
@andyschee942
7 ай бұрын
CI is the bare minimum. Most part of code review isn't even about things that broke. Also really depends on the quality of CI how reliable it is for detecting broken things.
@broadestsmiler
6 ай бұрын
@@andyschee942 Big vouch for the Scheester. Making sure that your code follows the proper standards (well documented with comments, proper usage of variables and functions, ensuring everything is clean etc.) is super integral to having great code. Bad code breaks, good code just works, great code works and can be understood and fixed easily in the future.
I just started my web development internship, and this is relatable af
waited so long for a new video lesgoooo!
😂😂😂😂😂"Talks in Intern" I can so relate
I wonder what the world of "structured" software engineering is like. I'm working in research and optimising/adapting ML algorithms but there's no code review, only results review, and no one else knows my code but me. I feel like I'm losing my mind with the freedom I'm allowed because I'll have 5 things I want to do but can only have time to do 1.
@traveller23e
7 ай бұрын
I work on legacy mainly, and it's a lot of "do exactly what was requested, and _only_ what was requested". Just today I got back from lunch to find several messages about how a PR of mine last week broke the release branch for everyone on another (parallel) team. This issue could be solved with a basic try-catch, but to be honest having looked through it further I don't know that we want to even make the change I was asked to make (and already implemented, albeit with a bug) due to subtle issues that are direct results of this plan.
@antoruby
7 ай бұрын
You need someone to help you with prioritizing the tasks, taking stakeholder impact into account
@rewrose2838
7 ай бұрын
@@traveller23eThis was nice to read. Sometimes I'm scared that I'm the only idiot that makes these kinda breaking changes.
@nam9221
7 ай бұрын
Im working as an intern in an ML research project, mostly just reporting results from different papers. I feel like ive hit a ceiling mathematically and my major isnt focused on ML its actually cybersecurity. How do you suggest I get better?
@traveller23e
7 ай бұрын
@@rewrose2838 Nah mate, people have broken release three times since I wrote that a week ago. As long as someone catches it before there's a real problem, you're fine.
This was too good!!!🤣🤣🤣🤣
LGTM, great work! If this was real I would cry at my desk.
YOOO my favorite youtuber uploaded, day made😍
@brnzhng
7 ай бұрын
LFFGG
The relatability in this😂
This is amazing, keep pausing the screen for the fantastic jokes all around >guys why is prod down it's literally 1 am
As a 28 year old whos never had a job I can confirm this is an accurate depiction of being an intern at a tech startup
@SkyArmysGeneral
7 ай бұрын
Bum
@surfingbilly9654
7 ай бұрын
Gigachad
@MrCmon113
3 ай бұрын
Keep it like that. Work sucks.
@DrDiabolical000
5 күн бұрын
@@SkyArmysGenerallife's tough and unpredictable. Be humble.
This video was my reason to subscribe
The only difference between this and “seniors” reviewing is that they nitpick even dumber things and constantly ask you what something does
@nooblal
7 ай бұрын
Knowing when and how to ask questions is knowledge too
@coherentpanda7115
7 ай бұрын
I don't mind nitpicking, it's all in how they conduct it. If they ask the right questions, like "Would it be better if we do XYZ? or "Are we able to do such and such on this line?" than they give you the opportunity to rebuttal or agree without them feeling like an ass, and you feeling like an idiot. Most good companies train on how to properly review PR's, and its a Senior priority to keep comments professional.
Love that red "Purchase" button. Such a subtle detail describing a dark pattern. But hey, if it makes money....😂
Got a senior in my team whom's process is probably similar because he only nitpicks on stuff while there's huge blatant bugs in plain sight
@coherentpanda7115
7 ай бұрын
It's that and also everyone has a Senior who just rubberstamps everything for their best friend that is filled with breaking changes, yet everytime they review your PR they go through it character by character nitpicking every detail with comments.
i love when speech is coming through my back left ear
ella ellaaa ehhh ehhhh shoutout rihanna ! good work on this one bro
This video had me in [speaking in intern] 😂. Idk why but it felt hilarious 😂.
"should be fine" now that's what i call high test confidence
i literally just had to review an MR on a project i worked on in the spring that a swe outside my team was trying to contribute to. It was terrifying
I have never related to a video more in my life 😭
"No description provided" smells like Sr.Engineer
spot on 🤣
great vid sir
hahaha this was good!
I am SO happy to have you posting again! Damn near pissed myself laughing LGTM!
@nicholast
7 ай бұрын
Glad you enjoyed it :)
This video is approved! LGTM
hes baaaaaaaack !!!
‘speaking in intern’ had me dying considering I’m still learning the language
+ Points on the theta numerology function, that was intense.
So good 😂
Is there are some more channels content like these? I really enjoy them😁
"senior intern" holy shit im dying
thanks for scaring me when I actually want to go intern lol
My left ear enjoyed this video 12.5% more than the right.
im boutta get that senior intern position
"Migrate literally everything to typescript."
HES BACKKKK
fwiw I wouldn't even look at a large pr like that 😂 usually they won't get looked at for a vvvv long time
Hey nick, nice to see youre making videos again! looking forward to seeing some more! p.s. you didnt show us part 2 of the date
Weather magician
Senior intern got me🤣
Intern => senior intern => intern manager....wow! great progress in career.
"speaking in intern" lmao
Lesson of the day, test before commit? lol
Shit, I had tears rolling on my cheeks from laughing. Nice one !
My lead is the definition of lgtm 😂 sometimes I’ll console.log some lyrics in my commits to see if they go through and they always do
"how do i run this?" Been there before 😭😭😭
I am near my internship completion, relatable af
WEATHER MAGICIAN
"Senior intern" looool never heard that one before
As someone that has no idea the nuance and genius of this skit, I kept waiting for the extra L in getUmbrelllas to pop up as the punchline. I assume since it didn't get pointed out that there are many little errors all throughout all the code in this and that's part of a running-gag-mini-side-punchline
My left ear really enjoyed this
😂😂😂senior intern
Congrats you are promoted to Senior Intern
hope mbrlla wont die after these reviews
SENIOR INTERN LMAO
What no BiG Time Rush at the end lol!!!
Meanwhile we just push 30 commits per day without description to main xD I love my team.
this is me as a full-timer
@nicholast
7 ай бұрын
I’m tryna get promoted to full timer
weather magician
Engineering Manager: guys who deleted 95% of the code?
imagine reviewing a pull request eith obfuscated code 💀
this was me last week for 1st week of work LOL
what was that first application where you drag and dropped a task from one list to another?
Resisting the temptation to rename all my variables using emojis... 😅
Same thing happened to me as a intern. You know what my senior co-worker did? Without any word he stood up from his chair, walked next to me, took my laptop and reviewed it in 2 seconds. Yeah, at that time it was my first experience with git, so I didn’t even remembered what happened 😂 Of course he was teaching me more later
Senior intern 🤣
1:01: using TypeScript and declare a new function with implicit any type parameter🤯
Ana left DotA to be a dev