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.
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.
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”.
Just use an opinionated code formatter in your per-commit automation. Fewer people get cut and noone gets sick with linting issues.
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.
You don’t have an autoformatter in your pre-commit hook? Why not?
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.
Can I integrate the linter with our HR management software to automate employee reviews?
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”.
But Shoggoth demands blood