6
Tounai
9h

I have been told to create smaller PRs, so now every big PR has a few other smaller ones associated. Hope my manager will be happy.

Comments
  • 6
    That's bullshit. Create smaller tickets. Then say, one pull request per ticket. That's the way.

    If he creates huge tickets, pull requests will be huge.
  • 2
    wtf... how ..... what..... huh??

    How the FUCK are you supposed to create smaller PRs?
  • 3
    @TrayKnots tickets can’t always be small, some pieces of work can’t be done with small changes without putting a mess
    I just like those big pieces of work.
  • 1
    @Tounai

    I agree, cannot always be small. But we have just moved the responsibility to whomever makes the tickets. If that person cannot make a small ticket, you cannot make a small pull request. That's how simple it is.
  • 2
    Tomorrow you'll scarp the rest of the big PR when you get a comment on the small one that changes everything 😉
  • 1
    @TrayKnots
    In fact I wrote the ticket.
  • 1
    @Tounai

    Kind of expected you to say this. The complain is upstream.

    Just tell whoever complained he should complain to the moron writing the big tickets and when that person asks who wrote it, turn around, put on a baseball cap, turn back and say: "That would be me."
  • 2
    big ticket -> small tickets

    PR for each small ticket

    not all tickets have a PR, but every PR has a ticket

    ---

    if something gets fundamentally changed there's no avoiding having to change a lot of code
  • 1
    ... the director of the department told me they wanted to refactor the code and they had a bunch of code debt, so whenever the lead kept giving me tickets to work on I would improve the code around it. then she called my PR a "red flag" because apparently I changed too much. MIXED MESSAGES. I just made the logging system more concise and informative (because I saw earlier the director was having trouble tracing data through the logs because info he needed wasn't being logged from the code) and removed some code duplications... zzzz. I don't get people
  • 0
    Most of my PR’s are big lol
Add Comment