this post was submitted on 27 Apr 2024
614 points (98.4% liked)

Programmer Humor

19594 readers
597 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
[–] [email protected] 58 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 36 points 6 months ago (3 children)

And then notice the spelling error.

[–] [email protected] 23 points 6 months ago

My spelling errors only come into existence after I commit.

[–] [email protected] 6 points 6 months ago

Just another free PR ;)

[–] Korne127 5 points 6 months ago (1 children)

Instant commit --amend + push --force

[–] [email protected] 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.

[–] [email protected] 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.

[–] [email protected] 4 points 6 months ago (1 children)

Wait what is the difference between the two?

[–] [email protected] 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.

[–] [email protected] 3 points 6 months ago

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