Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
Lock down the main and develop branch and enforce pull requests?
Every developer makes mistakes that can be caught in reviews -
NoMad141764yIf he's really a junior, he'd appreciate review. Tell him to make use of branches. And that'd you'd review his work and give him cookies if he does a good job.
-
nibor48774yWhy can anyone push to the main branch?
Don't do this, stick some proper permissions on the repo, and get PRs requiring at least 2 senior reviewers set up. -
Hazarth95014yWhat I did was wall to the guy who's code I was reviewing and told him:
"You're joking, right? There's no way I'm letting this pass, you can't X and Y... I left you comments, check it out because this is unacceptable"
True story, just a different language! They don't hate me and I have to say all code from them since was a lot more thought out... -
@butwhytho Still, proper procedure prevents dumb mistakes
What if a junior destroys his local branch and then accidentally force pushes to master? -
@butwhytho I feel this, small companies tend to forget smaller details. It sucks but the only thing you can do is trying and lock stuff down and push a standard of rules when handling repo's.
how do you tell a junior dev on your team there code is shit and please don't push to the main codebase without approval first,
without crushing them and and sounding like an ass cause there a good person just a n00b developer
question