Let him play in the legacy code. You can just hose him off later before letting him back into the office so he doesn’t track it everywhere.
Unfortunately if you let Junior play in legacy code once, it’ll learn some nasty habits and make more of it from scratch, usually when you’re trying to sleep.
deleted by creator
I don’t want that shit all over me though. That’s why I hired junior!
The leash is good unit testing.
Oh, he’ll just change the unit test if it fails.
Unit tests are there to get an @ignore annotation!
There’s also that long time senior dev who’s overly confident in their abilities and force pushes production breaking code directly to master.
Not for long there isn’t.
deleted by creator
Neither does the senior
Anymore
me when first starting out at a job commenting everything I can
VS
me a couple years in completely lost because I never updated the comments and now none of them make any sense whatsoeverCommenting well is a highly advanced skill. I generally prefer no comments on code since it’s less likely to confuse people and I’ll merrily purge auto-doc comments and anything like
// getId() returns an id
That comment has negative value.
Make him unit test it all and then they can do changes that don’t break.
Often enough, the old code is so badly intertwined that it’s impossible to actually test. Those are the moments where all you can do is nuke it from orbit.
Why is the junior dev tied to the other one’s penis?
Bro that’s not where the penis is located on the body
Hey, I don’t body shame
