265
submitted 3 days ago* (last edited 3 days ago) by ulterno@programming.dev to c/programmer_humor@programming.dev

Until he actually had to use it.

Took 2 hours of reading through examples just to deploy the site.
Turns out, it is hard to do even just the bash stuff when you can't see the container.

you are viewing a single comment's thread
view the rest of the comments
[-] frezik@midwest.social 36 points 3 days ago

Normally, you don't want to commit code unless it's been at least minimally tested, and preferably more than that.

All the CI's, however, force a workflow where you can only test it by committing the code and seeing if it works. I'm not sure how to fix that, but I see the problem.

[-] Ajen@sh.itjust.works 6 points 2 days ago

Line the other commenter said, there's nothing wrong with committing temp/untested code to a feature branch as long as you clean it up before the PR.

[-] frezik@midwest.social 4 points 2 days ago

There are issues that come up in niche cases. If you're using git bisect to track down a bug, a non-working commit can throw that off.

[-] Ajen@sh.itjust.works 8 points 2 days ago* (last edited 2 days ago)

You might have misunderstood what I meant by "clean up before the PR." None of the temp commits should end up in the main branch, where people would be bisecting.

load more comments (10 replies)
this post was submitted on 13 Nov 2024
265 points (97.8% liked)

Programmer Humor

19564 readers
1036 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