477
submitted 11 months ago by qaz@lemmy.world to c/programmerhumor@lemmy.ml
you are viewing a single comment's thread
view the rest of the comments
[-] SpaceNoodle@lemmy.world 6 points 11 months ago

I will cut someone for leaving trailing whitespace or mixing spaces and tabs, though.

[-] nickwitha_k@lemmy.sdf.org 15 points 11 months ago

Just use an opinionated code formatter in your per-commit automation. Fewer people get cut and noone gets sick with linting issues.

[-] TheYear2525@lemmy.world 3 points 11 months ago

Can I integrate the linter with our HR management software to automate employee reviews?

[-] nickwitha_k@lemmy.sdf.org 2 points 11 months ago

Sure but I'm not sure that most HR management software companies are going to be keen on handing over source for you to review.

5/7 Software package is pure garbage. Tim forgot to remove his TODO comment from line 1052 of main and there are three instances of inconsistent indentation in the API module. Therefore, our automation pipeline has marked our own employees as "needs improvement".

[-] fushuan@lemm.ee 3 points 11 months ago

It's annoying to be the weird one that rewrites all the code on the files I touch because I bothered to press format. Then someone forgets to pull before changing the code and suddenly the merge is not fast forward and the conflict resolution is a mess. It's not that big of a deal, for me a t least, but when i format the code it's 3 other dumbasses that get conflicts, and sometimes I just can't bother enough to fix their issues because I took care of formatting it once.

Nowadays I use an opinionated linter, format my stuff and call it a day.

[-] Eufalconimorph@discuss.tchncs.de 3 points 11 months ago

You don't have an autoformatter in your pre-commit hook? Why not?

[-] nickwitha_k@lemmy.sdf.org 1 points 11 months ago

I loathe linter errors. If you know that I did it wrong and how, just fix it and stop wasting everyone's time. That's why I'm in favor of opinionated formatters in the per-commit automation. No point in wasting everyone's time and making silly merge conflicts possible.

[-] SpaceNoodle@lemmy.world 2 points 11 months ago

But Shoggoth demands blood

[-] souperk@reddthat.com 1 points 11 months ago

last week I had misconfigured my auto-format and it was leaving commas and whitespaces. The amount of "WHAT HAVE YOU DONE???" comments I got were of the chart.

There was a linter in place, I literally could not merge unless the issues were fixed, yet people felt compelled to point them out.

this post was submitted on 15 Oct 2023
477 points (95.6% liked)

Programmer Humor

32060 readers
1834 users here now

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

Rules:

founded 5 years ago
MODERATORS