• @[email protected]
    link
    fedilink
    848 months ago

    I disagree.

    Be…before I go. Could you approve all my experimental PRs?

    Real story, I have a branch that’s been open for four years at my company to add support for nested postgres transactions. It works flawlessly… but we, the senior devs, are uncertain if it’s a power that would be used responsibly by the juniors. If I’m going to walk into the light, I’m going to make sure there’s a badass explosion behind me.

    • @[email protected]
      link
      fedilink
      118 months ago

      I’ve got one for a vscode feature (middle mouse click for go-to definition) that I want but the maintainers don’t think it would be used.

      • @Opisek
        link
        88 months ago

        I’m in the niche of niches by using vsc with a vim plugin while being a dvorak user. I rely on vim’s langmap feature to get anywhere, but people implementing vim emulators blissfully ignore accessibility like that. So I went and implemented langmap in vsc’s vim plugin myself. It has minimal intervention into the existing codebase and a bunch of other people have been wishing for this for years. Yet, when it comes to merging… Silence.

        • @[email protected]
          link
          fedilink
          48 months ago

          Can’t you just open source your fork? I mean yeah it does suck but still. I’ve been in the same place and it does suck when they won’t take your PR.

      • @IHateReddit
        link
        18 months ago

        I try to use middle click for definition in vscode so often because you can do that in jetbrains IDEs…

  • @[email protected]
    link
    fedilink
    49
    edit-2
    8 months ago

    You haven’t written your unit tests and you have 675 compiling errors left. It’s time to sleep grandpa

  • @[email protected]
    link
    fedilink
    32
    edit-2
    8 months ago

    That’s why I’ve got my IDE configured to make a commit and push for every single ctrl-s.

    And one more thing, I’m not going to squash before my final PR.

    EDIT:

    For those of you interested, here’s my gitconfig alias to help with this workflow:

    [alias]
        ctrl-s-commit-push="!f() { count=$(cat count.txt); git add .; git commit -m \"$(date): commit $count\"; git push;}; f"
    

    IMPORTANT: For those who are downvoting, yes, my entire comment is sarcasm, lol! Do not ever do anything like this in a shared repo, ever! If you actually do this in a shared repo, your access will likely be revoked the moment someone sane and competent realizes what you’re doing.

    Addendum: In that alias, I would’ve used left angle bracket instead of cat, but apparently lemmy scrubs those. I even tried the entity for it, but no go for either.

    • @AstridWipenaugh
      link
      98 months ago

      Your sarcasm is brutal! That’s sarcasm, right? RIGHT?!

    • JelloeaterA
      link
      English
      48 months ago

      Squash and Force Push. Your whole life in one line.

    • Ignotum
      link
      38 months ago

      Combine that with my pre-push hook that runs linting and tests, which takes about 10 minutes, and you’re gonna have a good time

      I’ve long ago disabled the tests and only run linting now, i’m not a machochist

    • ntzm [he/him]
      link
      fedilink
      English
      38 months ago

      Wouldn’t that mean you just have shit tons of commits? What about the commit message?