10

I hate people who merge the PR while the others are in the middle of the fucking code review

Comments
  • 2
    Merge their head with your keyboard. Or theirs.
  • 2
    Slam their knuckles with a hammer while they're typing.
  • 1
    How does your repo admin allow this?
  • 3
    Don't you have restrictions so as to allow the merging?
  • 1
    Usually this is a sign of not having a PR / review workflow.

    Workflow consists of statuses, which should be visible.

    Without a workflow, reviewing is really for nuts, since there is no coordination and things can go wrong bad TM.
  • 1
    It sounds like the PR settings on the repo weren't adequately configured or that your team didn't have a set code review workflow.
    So unless the person who merged it has admin rights and abused that, then it's the fault of the repo owner / admin for not configuring PR merge conditions according to whatever was agreed within your team.
  • 2
    slap em, with yo pee pee, if you is a woman then slap em with yo chichis.
  • 0
    I tried to tag everyone but I couldn't. All teams have the settings done properly. It's the abuse of power the problem.
  • 0
    @AleCx04 my pee pee is not as big as my balls that as a matter of fact are also not big enough since they didn't solve the problem once I put them on the table :( tldr; my pee pee is tiny
Add Comment