• ngn@lemy.lolEnglish
    6·
    9 hours ago

    i don’t understand it, am i missing some context here?

    • pixelscript@lemm.eeEnglish
      17·
      8 hours ago

      The meme format implies she catfished you with the promise of “Netflix and chill” at her house only to pull a gun on you.

      In particular, she wants you to review and merge that goddamn pull request she made to your open source project repo two months ago that finally fixes that one really annoying bug.

  • mspencer712@programming.dev
    75·
    1 day ago

    Remove these blank lines.

    I’m not seeing unit tests for this.

    Unnecessary comment.

    BLAM

    Ow! Also, this could’ve been a smaller calibur.

  • steventhedev@lemmy.world
    36·
    1 day ago

    I have either written or gotten a variant of every single one of these comments 🫠:

    Please include the JIRA task in the commit title.

    Did you run any manual testing?

    Where’s the PRD link in the commit message?

    Can you please split this into multiple smaller commits?

    Can you combine these two commits?

    Did you email Jon about this because he’s working on that project with Sarah and you might be duplicating efforts.

    This should be named BarFoo instead of FooBar.

    Why aren’t you using CorporateInternalLib16 that does 90% of this?

    Why aren’t you using ThirdPartyPaidLibByExEmployee?

    Why aren’t you using StandardLib thing you forgot existed?

    All our I/O should be async.

    All our hot loop code needs to be sync.

    This will increase latency of NonCoreBusinessFlow by 0.01%. can you shave some time off so we can push in feature B also?

    Please add a feature flag so we can do gradual rollout.

    What operational levers does this have?

    Lgtm - just address those comments

      • frezik@midwest.social
        1·
        5 hours ago

        Mostly, yes.

        I’d like to find a better way to phrase "why aren’t you . . . " questions. It carries an accusatory tone in text, even if you don’t intend that. The answer is almost invariably going to be either “I didn’t know it existed” or “because reason X”. Neither case justifies the accusatory tone. Maybe if the “I didn’t know it existed” answer was something so basic that they really should have known it existed, but probably not even then.

        • Thinker@lemmy.world
          1·
          20 minutes ago

          My preferred variation of this is to make it an open question that leaves them in the position of authority, and assumes that they made a deliberate decision.

          For example, instead of “Why aren’t you using StandardLib that does 90% of this?”, I would try “Could this be achieved with StandardLib? Seems like it would cover 90% of this”.

  • SavvyWolf@pawb.socialEnglish
    19·
    24 hours ago

    “Did you run the formatter on this?”

    Bonus points if it’s python code and nowhere in the docs does it say which of the many formatters to use.

      • bob_lemon@feddit.org
        4·
        17 hours ago

        I’m fond of ruff lately. Pretty much the same as black, but it just comes with the linter instead of being separate tools.

        • Gamma@beehaw.orgEnglish
          2·
          7 hours ago

          👀 I’d fine-tuned (copied and pasted) a pylint setup that had some of black, what are the differences?

          Edit: it’s so much faster than black, I’m switching

  • whoisearth@lemmy.ca
    5·
    23 hours ago

    First problem is she’s expecting a right boob like me to review her pull request. Second problem is I don’t give a shit.