When You Ask the Intern to Review Your Code

Поделиться
HTML-код
  • Опубликовано: 2 дек 2024

Комментарии • 366

  • @HoloTheDrunk
    @HoloTheDrunk Год назад +3133

    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
      @weak1ings Год назад +122

      And then never getting merged until it is inevitably closed!

    • @rewrose2838
      @rewrose2838 Год назад +77

      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
      @AnibalAlvarez Год назад +22

      The best python package of my life was never even reviewed

    • @augustofotino523
      @augustofotino523 Год назад +3

      This gave me PTSD

    • @tilmanrotationalinvariant2257
      @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...

  • @AwesomeDwarves
    @AwesomeDwarves Год назад +718

    That feeling of "He wants me to review *his* code?" is truly universal for software devs

  • @peterlyon185
    @peterlyon185 Год назад +714

    Intern manager had me dying 💀

  • @TheAkiller101
    @TheAkiller101 Год назад +1649

    switching variable colour to color has made it 20% efficient saving millions of dollars in cloud cost. keep up the great work !

    • @AryanGNU
      @AryanGNU Год назад +7

      Lmao chuckled so hard 😂😂😂

    • @triplezgames3882
      @triplezgames3882 Год назад +49

      Actually, "colour" is 20% less efficient than "color",
      but "color" is 16,67% more effective than "colour"

    • @magicmulder
      @magicmulder Год назад +17

      You can actually put 10% of the code in the filename. Huge space saver.

    • @hnazmul
      @hnazmul Год назад +1

      😂😂😂😂

  • @michaelburton9420
    @michaelburton9420 Год назад +1512

    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
      @AnibalAlvarez Год назад +16

      There are some words in that hudge fact

    • @KristianTheDesigner
      @KristianTheDesigner Год назад +10

      As a frontend dev who started just 6 months ago i need to say that this comment made me calm 😁 Thanks

    • @xgcwrought3346
      @xgcwrought3346 Год назад +1

      Omg is this what I get to look forward to?

    • @liloruf2838
      @liloruf2838 Год назад +21

      True! The best is when seniors just explain everything without me having to ask😂😂😭

    • @sypwer
      @sypwer 6 месяцев назад

      If that's honestly the senior tension I might not ask questions ever

  • @cody_codes_youtube
    @cody_codes_youtube Год назад +298

    Ohhhh my god. Sooo many Easter eggs in this video. This had to take so much time and also worth every second!

    • @kevinbrandon1856
      @kevinbrandon1856 6 месяцев назад +1

      Please don’t use the name of God in vain!

    • @meek0076
      @meek0076 3 месяца назад

      ​@@kevinbrandon1856god is a sigma frfr

  • @JTBanks
    @JTBanks Год назад +681

    This has no business being this good!! Insane how much work put into this, truly appreciate you.

    • @nicholast
      @nicholast  Год назад +21

      glad you enjoyed it

    • @adityasuryawanshi3263
      @adityasuryawanshi3263 Год назад +2

      ​@@nicholastwhat is the name of that utility at the start of the video where you are managing all the tasks, looks cool?

    • @marjo9952
      @marjo9952 Год назад +2

      @@adityasuryawanshi3263 "Trello" I'm assuming

  • @jondoty
    @jondoty Год назад +272

    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
      @coherentpanda7115 Год назад +17

      That's an every software developer experience, including for seniors.

    • @NevosLP
      @NevosLP Год назад +3

      Has happened so many times to me...

    • @marc-andrefortin
      @marc-andrefortin 11 месяцев назад +1

      Classic

    • @maryammoghaddas768
      @maryammoghaddas768 5 месяцев назад +1

      Unfortunately I have been the one creating this scenario for a new hire. It sucks on the senior/mid side too. After several cases of this, I tried to iron out the details before giving the task and that helped tremendously.

  • @max3446
    @max3446 Год назад +984

    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
      @matteosposato9448 Год назад +97

      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
      @hunter-tm2kl 9 месяцев назад

      ​@@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
      @HeatFort 9 месяцев назад

      @@matteosposato9448probably still in school so they were waiting to hire him

    • @marcialabrahantes3369
      @marcialabrahantes3369 8 месяцев назад

      ​@@matteosposato9448I've seen that for students that are going into Master's program

    • @nootanwait2358
      @nootanwait2358 6 месяцев назад

      ​​@@matteosposato9448I did 3 paid internships for the same company - during the student vacations each year in university. So if they're still full time studying then it's not necessarily exploitative

  • @otisroot
    @otisroot Год назад +123

    Not only did I review my mentor’s PR, I REJECTED it

    • @jjinendra3
      @jjinendra3 Месяц назад

      Lol what'd he say?😂😂

  • @MrAustonpowers
    @MrAustonpowers Год назад +59

    I recently got put on a new project at work and this is way too relatable / anxiety inducing! Well done, sir!

  • @za_wavbit
    @za_wavbit Год назад +96

    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
      @andyschee942 Год назад +29

      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
      @broadestsmiler Год назад +1

      @@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.

  • @bluesteel1
    @bluesteel1 Год назад +53

    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
      @angelg3642 9 месяцев назад

      Naaah you legit gotta do that 😂😂😂😂😂😂. The smarter it sounds the more impressed the HR's are. It's legit comedy sometimes

    • @lockaltube
      @lockaltube 6 месяцев назад

      Sounds more like "improved performance", because red is faster!

    • @Mempler
      @Mempler 3 месяца назад

      ​@@angelg3642the less people understand, the more they think that you know your stuff

  • @ogmoiz
    @ogmoiz Год назад +232

    after starting my first internship i can finally relate to all the intern memes and confirm they are 100% accurate

  • @minnie-piano3969
    @minnie-piano3969 Год назад +138

    0:43 you forgot the holy title of INTERN CEO

    • @nicholast
      @nicholast  Год назад +74

      chief intern officer

    • @d_sanu
      @d_sanu Год назад +10

      That's a CIO... 😂 not CEO

    • @armtdawg99
      @armtdawg99 10 месяцев назад +6

      Chief Executive Intern

  • @twilightgeneral777
    @twilightgeneral777 Год назад +15

    Congrats on the promotion to Double Super Intern: First Class!

  • @HM-ch
    @HM-ch Год назад +6

    I showed your video to the Senior Intern on our team and he confirmed that this is how it goes.

    • @keyeslord
      @keyeslord 6 месяцев назад

      You guys have a senior intern??

  • @Sh4ndes
    @Sh4ndes Год назад +18

    I just started my web development internship, and this is relatable af

  • @Ecann72
    @Ecann72 Год назад +5

    IVE MISSED THESE VIDEOS

  • @limeisrichard
    @limeisrichard Год назад +1

    i love when speech is coming through my back left ear

  • @sandupaegodage8163
    @sandupaegodage8163 Год назад +3

    "No description provided" smells like Sr.Engineer

  • @Nsimi03
    @Nsimi03 Год назад +2

    love it man! Keep dropping.

  • @natashajchen
    @natashajchen Год назад +1

    video editing attention to detail goes craaazy !!

  • @nullObject_
    @nullObject_ 5 месяцев назад

    00:24 "He wants me to review his code?"
    Every SE has experienced this feeling once

  • @GuagoFruit
    @GuagoFruit Год назад +113

    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
      @traveller23e Год назад +21

      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
      @antoruby Год назад +6

      You need someone to help you with prioritizing the tasks, taking stakeholder impact into account

    • @rewrose2838
      @rewrose2838 Год назад +7

      ​@@traveller23eThis was nice to read. Sometimes I'm scared that I'm the only idiot that makes these kinda breaking changes.

    • @nam9221
      @nam9221 Год назад +2

      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
      @traveller23e Год назад +4

      @@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.

  • @robottrainer
    @robottrainer Год назад +2

    + Points on the theta numerology function, that was intense.

  • @delevoxdg
    @delevoxdg Год назад +10

    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.

  • @NathanLuMax
    @NathanLuMax Год назад +21

    Bro I relate to this so hard 😂

  • @NotYoel
    @NotYoel Год назад +4

    loved the video. keep dropping more 👍

  • @sibusisomfana6388
    @sibusisomfana6388 Год назад +15

    😂😂😂😂😂"Talks in Intern" I can so relate

  • @striderstache99
    @striderstache99 Год назад +4

    Love that red "Purchase" button. Such a subtle detail describing a dark pattern. But hey, if it makes money....😂

  • @joehaynes7092
    @joehaynes7092 Год назад +15

    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.☠️

  • @GeneralKenobi69420
    @GeneralKenobi69420 Год назад +51

    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
      @SkyArmysGeneral Год назад +7

      Bum

    • @surfingbilly9654
      @surfingbilly9654 Год назад +8

      Gigachad

    • @MrCmon113
      @MrCmon113 10 месяцев назад

      Keep it like that. Work sucks.

    • @DrDiabolical000
      @DrDiabolical000 7 месяцев назад +2

      ​@@SkyArmysGenerallife's tough and unpredictable. Be humble.

    • @keyeslord
      @keyeslord 6 месяцев назад

      Being privileged enough to never work is crazy

  • @Claude_Developer
    @Claude_Developer 4 месяца назад

    Great work! love your content. Keep it up!

  • @cokikillide5855
    @cokikillide5855 Год назад +166

    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
      @Haise-san Год назад +39

      That doesn't sound like a "senior" dev, but what do we juniors know lmao, maybe he was into something

    • @jakobullmann7586
      @jakobullmann7586 Год назад +27

      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.

    • @siliconhawk
      @siliconhawk 10 месяцев назад

      ​@@Haise-sanhe probably was on to something he didn't become a senior engineer just like that.
      right guys ?

    • @MrCmon113
      @MrCmon113 10 месяцев назад +27

      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
      @angelg3642 9 месяцев назад +3

      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 ????

  • @bilgecimen4243
    @bilgecimen4243 Год назад

    This made my day, great content!

  • @mctv2-randomness355
    @mctv2-randomness355 Год назад +2

    YOOO my favorite youtuber uploaded, day made😍

  • @vladvlad2289
    @vladvlad2289 11 месяцев назад

    1:15 If I ever see that in a PR, I'm going to church.

  • @ilearncode7365
    @ilearncode7365 Год назад +8

    The only difference between this and “seniors” reviewing is that they nitpick even dumber things and constantly ask you what something does

    • @nooblal
      @nooblal Год назад +1

      Knowing when and how to ask questions is knowledge too

    • @coherentpanda7115
      @coherentpanda7115 Год назад +3

      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.

  • @DjSquatter
    @DjSquatter 8 месяцев назад

    Shit, I had tears rolling on my cheeks from laughing. Nice one !

  • @davidmwakonya3324
    @davidmwakonya3324 Год назад +5

    The relatability in this😂

  • @darkin1484
    @darkin1484 Год назад +3

    Engineering Manager: guys who deleted 95% of the code?

  • @topsykretts2264
    @topsykretts2264 11 месяцев назад +1

    Always better to ask questions. Just make sure it’s the right time and setting. Know how to read the room.

  • @nojustno2526
    @nojustno2526 Год назад +5

    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

  • @notflash11
    @notflash11 2 месяца назад

    Intern manager🤣🤣

  • @nxlier6390
    @nxlier6390 2 месяца назад +1

    Senior intern😂

  • @WelcomeBub
    @WelcomeBub Год назад +1

    LGTM, great work! If this was real I would cry at my desk.

  • @Dnserror88
    @Dnserror88 4 месяца назад

    3.5 YOE as a front-end dev and finally feel comfortable reviewing seniors' code and actually providing meaningful feedback and/or improvements lol.

  • @AeonFM
    @AeonFM 9 месяцев назад

    My left ear really enjoyed this

  • @MrDavidRaw
    @MrDavidRaw Год назад +1

    ella ellaaa ehhh ehhhh shoutout rihanna !
    good work on this one bro

  • @TheGrimravager
    @TheGrimravager Год назад

    This is amazing, keep pausing the screen for the fantastic jokes all around
    >guys why is prod down it's literally 1 am

  • @Gamer_with_knowledge
    @Gamer_with_knowledge Год назад

    Was waiting for your videos :)

  • @dragonarc_gaming
    @dragonarc_gaming Год назад

    When the senior is drunk and the intern is delusional. Good context I should say XD

  • @harshilshah980
    @harshilshah980 Год назад +1

    ‘speaking in intern’ had me dying considering I’m still learning the language

  • @prasenjitsutradhar3368
    @prasenjitsutradhar3368 Год назад +1

    Intern => senior intern => intern manager....wow! great progress in career.

  • @faresk3168
    @faresk3168 Год назад +1

    Welcome back man!!

  • @mlevvy96
    @mlevvy96 9 месяцев назад

    "yeaaaahhh should be fine" half of the time ends in disaster XD of course if it's merged without in-depth code review

  • @bananesalee7086
    @bananesalee7086 Год назад +9

    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
      @coherentpanda7115 Год назад

      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.

  • @tonyjames1980
    @tonyjames1980 4 месяца назад

    That was really funny, thank you

  • @hjkfyt3305
    @hjkfyt3305 9 месяцев назад +1

    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

  • @StinkyCatFarts
    @StinkyCatFarts 6 месяцев назад

    Send it back to the senior and tell him to code like a senior

  • @thereasonableprogrammer4921
    @thereasonableprogrammer4921 10 месяцев назад

    The “No description provided” on the pr hit deep 😂

  • @souravh7637
    @souravh7637 6 месяцев назад +1

    "Speaking Intern " --- All can relate ig🤣

  • @SuperElephant
    @SuperElephant Год назад

    My left ear enjoyed this video 12.5% more than the right.

  • @abdiraqiibmohummed9781
    @abdiraqiibmohummed9781 Год назад +2

    hes baaaaaaaack !!!

  • @johnallentech
    @johnallentech Год назад

    This video was my reason to subscribe

  • @taeahn6065
    @taeahn6065 5 месяцев назад +1

    I have no idea how to code. The video was good i guess.
    Why am I here.
    I'll go back to school in order to understand this video

  • @dapyr
    @dapyr Год назад +1

    Weather magician

  • @tarn84
    @tarn84 Год назад +1

    ayeee he is back with the quality content

  • @gordonduan2415
    @gordonduan2415 Год назад +1

    I have never related to a video more in my life 😭

  • @xellestar
    @xellestar Год назад

    "should be fine" now that's what i call high test confidence

  • @easternplatypus
    @easternplatypus Год назад +1

    my left ear liked this video 👍

  • @GainsGoblin
    @GainsGoblin Год назад +2

    "senior intern" holy shit im dying

  • @andrewdddo
    @andrewdddo Год назад

    waited so long for a new video lesgoooo!

  • @johndevnoza4223
    @johndevnoza4223 3 месяца назад

    "Senior intern" i lost it! :Ddddd

  • @jasonpittman7853
    @jasonpittman7853 4 месяца назад

    I’m an intern and my boss just told me to have the other intern run the PR. Godspeed codebase two interns have invaded.

  • @QuransGems-zd1jd
    @QuransGems-zd1jd 4 месяца назад

    2:54 was the best!

  • @avalerionbass
    @avalerionbass 6 месяцев назад

    Congratulations! You've been promoted to CEI, Chief Executive Intern.

  • @evilcorp.7288
    @evilcorp.7288 Год назад

    does anyone know what software is that 0:02 (task schedular looking thing)

  • @stevengongg
    @stevengongg Год назад

    great vid sir

  • @doorey2
    @doorey2 Год назад +1

    "Migrate literally everything to typescript."

  • @huynguyenq7333
    @huynguyenq7333 7 месяцев назад

    "speak in intern" part make me 😂

  • @VictorZheng231
    @VictorZheng231 Год назад +1

    Another banger 🔥🔥🔥

  • @Mohit-wb5cv
    @Mohit-wb5cv 5 месяцев назад

    weather magician

  • @csy897
    @csy897 Год назад

    LGTM let's go to mars (and never come back)

  • @IvanMelnikov
    @IvanMelnikov Год назад

    Man still cracking up over the PR title :D

  • @thomasn5726
    @thomasn5726 Год назад

    This video is approved! LGTM

  • @marcozhang7685
    @marcozhang7685 Год назад

    "speaking in intern" lmao

  • @bedro_0
    @bedro_0 Год назад

    bro looks like if two set violin had a child

  • @tiimmng
    @tiimmng Год назад +1

    fwiw I wouldn't even look at a large pr like that 😂
    usually they won't get looked at for a vvvv long time

  • @aressilver1951
    @aressilver1951 6 месяцев назад

    is nit a nitpick?

  • @shubhamdesai7730
    @shubhamdesai7730 Год назад +1

    Is there are some more channels content like these? I really enjoy them😁

  • @antikokalis
    @antikokalis Год назад

    I don't understand @3:19 Didn't he only leave comments? How was this damage done by comments?

    • @aleksandrapetela4858
      @aleksandrapetela4858 8 месяцев назад

      its the senior engineer reviewing the interns pr this time

  • @jojox1904
    @jojox1904 Год назад +2

    I accidentally took down one of our clients websites today for like an hour because I merged into master instead of testing 😭

  • @akashchhabra805
    @akashchhabra805 Год назад

    Hey weather magician, what's the name of tool you are using to track tasks at the beginning at 0:02 sec

    • @bastian5861
      @bastian5861 8 месяцев назад

      i know its been 6 months but its trello

  • @flamingsparrow8857
    @flamingsparrow8857 Год назад

    WEATHER MAGICIAN

  • @GnomeEU
    @GnomeEU Год назад

    Meanwhile we just push 30 commits per day without description to main xD
    I love my team.

  • @killdhero
    @killdhero Год назад

    What no BiG Time Rush at the end lol!!!

  • @Fremmy
    @Fremmy 4 месяца назад

    3:25 onwards is literally what happened with Cloudflare this past week.

  • @salal_guitar5583
    @salal_guitar5583 5 месяцев назад

    Lmao @ all the CI checks passing 😂