this post was submitted on 21 Oct 2023
725 points (98.3% liked)

Programmer Humor

32397 readers
1098 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS
 
top 26 comments
sorted by: hot top controversial new old
[–] [email protected] 83 points 1 year ago (2 children)

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] 11 points 1 year ago (2 children)

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.

[–] [email protected] 8 points 1 year ago (1 children)

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] 4 points 1 year 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.

[–] [email protected] 1 points 1 year ago (1 children)

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

[–] [email protected] 1 points 1 year ago

Give this PR some love then. It works and it's magical but they don't seem interested in merging it in. https://github.com/microsoft/vscode/pull/154465

[–] [email protected] 6 points 1 year ago* (last edited 1 year ago)

Godspeed my friend. 10/10 I would use that irresponsibly.

[–] [email protected] 48 points 1 year ago* (last edited 1 year ago) (1 children)

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

[–] [email protected] 13 points 1 year ago
[–] [email protected] 32 points 1 year ago* (last edited 1 year ago) (5 children)

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.

[–] [email protected] 9 points 1 year ago

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

[–] [email protected] 9 points 1 year ago

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

Pure evil.

[–] [email protected] 4 points 1 year ago

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

[–] [email protected] 3 points 1 year ago (4 children)

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

[–] [email protected] 7 points 1 year ago

"Hit Ctrl-S"

[–] [email protected] 2 points 1 year ago

I'd expect the poo emoji at that point.

[–] [email protected] 2 points 1 year ago

The more the merrier. I edited my comment above to show my git alias for comments :)

[–] [email protected] 0 points 1 year ago (2 children)
[–] [email protected] 3 points 1 year ago

That sounds just awful. But ok.

[–] [email protected] 1 points 1 year ago

I meant whatever works for you, but if a coworker did that I would be fuming

[–] [email protected] 3 points 1 year 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

[–] [email protected] 8 points 1 year ago

*commits

ERROR: There are 5000 conflicts, please resolve them before pushing

[–] [email protected] 2 points 1 year ago

It's too late.

[–] [email protected] 1 points 1 year ago (1 children)

I must be dumb, I don't get it

[–] [email protected] 3 points 1 year ago (1 children)

Git. The joke is they have stuff locally they want to push to a remote

[–] [email protected] 1 points 1 year ago

And make the production down.