this post was submitted on 27 Apr 2024
615 points (98.1% liked)

Programmer Humor

19548 readers
1081 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS
 
you are viewing a single comment's thread
view the rest of the comments
[–] LostXOR@fedia.io 59 points 6 months ago (2 children)

Gotta review the 5 line PR ten times just to make absolutely totally sure there's nothing wrong with it before submitting it.

[–] perviouslyiner@lemmy.world 37 points 6 months ago (3 children)

And then notice the spelling error.

[–] YIj54yALOJxEsY20eU@lemm.ee 23 points 6 months ago

My spelling errors only come into existence after I commit.

[–] Dhs92@programming.dev 6 points 6 months ago

Just another free PR ;)

[–] Korne127@lemmy.world 5 points 6 months ago (1 children)

Instant commit --amend + push --force

[–] omidmnz@programming.dev 6 points 6 months ago (1 children)

Prefer --force-with-lease. It doesn't make a difference in this scenario, but it's a safer habit/default.

[–] xmunk@sh.itjust.works 3 points 6 months ago (1 children)

There is no reason to force push anymore - force with lease is always preferable. So said, a frequent rebaser.

[–] rufus@lemmy.sdf.org 4 points 6 months ago (1 children)

Wait what is the difference between the two?

[–] xmunk@sh.itjust.works 11 points 6 months ago

Force with lease will send up what your local thinks the most recent commit on the branch was and the host will reject the push if it knows of a more recent commit on that branch. Basically, it saves you from overwriting a commit you didn't realize existed.

[–] SkyeStarfall@lemmy.blahaj.zone 3 points 6 months ago

Don't want to be the cause of a CVE now, do we?