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
-
ddephor45125yTell 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. -
One of my pull requests was once blocked because CI pipeline failed. Why? Linter issue. What? Linter throws no errors on my machine.
Turns out some twat pushed to master on an entirely unrelated part of the codebase. Likely someone forgot to prevent master pushes on that particular repository.
I pushed the lint fix straight to master with the message "If you push shit to master, I push shit to master as well" (this is a quote).
Rebased local branch on master, pushed to rerun CI and voila, runs through fine and gets merged to master after peer review. -
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 ! 😂
-
anolis3735y@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?
-
saas13044yEven 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. -
@mentions That actually seems like a sensible approach to rudimentary 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