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
-
ddephor43635yTell him what's wrong and create a VCS guideline, so there are written requirements for correct usage.
If possible, enforce the technically checkable requirements on the server with commit-hooks.
If someone here would commit this way, he would be lynched immediately, and justifiably so. -
Two thoughts:
1. Does he own his fuckup?
That's really the key here. If they're capable of acknowledging their failure and remedying it, that's a positive.
2. Is the rest of the team capable of reviewing his code?
This is a spot I've been in a few times, assigned a team that was underskilled to such a degree none of them could review my code, so issuing the pull request to trunk would be ceremony.
Either way, conversations should be had. Tests should be written. -
@SortOfTested
I use PRs even in my solo private repos.
Mostly because having the merge history and "revert PR" button is much more convenient than finding all the commits you want to revert by hand if one feature deployment goes wrong. -
@bittersweet
I do single feature squash commits and use git revert. Merge commits are just noise to me. -
We have one person who will put various things like this including 'lulz' 'broke shit'
Even more infuriating when their is no ticket number. -
Wait what? How do you guys cherry pick these commits? Like 'Oh', 'yeah' cherry pick me ! 😂
-
anolis3555y@nitwhiz what do you even mean by this comment "Autism" wtf dude? Ableist much? Kind of offensive to people that are doing a really good job that have the condition?
-
saas12735yEven i didn't commit to master when i wasn't using version control. I used to copy the master and make changes into the copy (cuz too scared to fuck up)
P.S. I was just getting started with coding in middle school and we didn't know about version control.
Related Rants
My lead keeps pushing commits to master. His commit messages vary from: no message, yeah, and yup.
and yea, some of the build break master.
Makes me just wanna die sometimes when digging through our commit history to figure out when a bug was introduced.
rant
master
commit history
yeah
github