undefined

points

[-]

Hmm, in every team I've been in (only 3 tbf) we almost all followed the "nit" approach for PRs.

    nit: this could be changed to XYZ

    we should use XYZ here

where it was understood nits could be ignored if you didn't feel it was an urgent thing vs a preference.

by zoogeny16 hours ago|

parent|

[-]

It's worth noting that this is a kind of different "nit" than something that might be attached to a line of code. Like, someone might "nit" using a bunch of if statements where a switch statement might work, or if someone uses a `for each` where a `thing.map` would do.

What I am describing would be something higher level, more like a comment on approach, or an observation that there is some high-level redundancy or opportunity for refactor. Something like "in an ideal world we would offload some of this to an external cache server instead of an in-memory store but this is better than hitting the DB on every request".

That kind of observation may come up in top-level comment on a code review, but it might also come up in a tech review long before a line of code has been written. It is about extending that attitude to all aspects of dev.

by lazyasciiart10 hours ago|

parent|

[-]

I had someone reject my code that improved/regularized half a dozen instances of a domain object we had, where they were showing up in code paths I cared about. He said there’s dozens of these, don’t submit this unless you fix them all.

by YZF9 hours ago|

parent|

[-]

I had something similar but convinced the other person the rest of the work can be done later. Then the person went ahead and did it despite the other instances having no use/value. Go figure. I guess having consistency has some value to argue the other side. I tend to be extremely flexible in terms of allowing different ways of doing things but some seem to confuse form with function insisting on some "perfection" in the details. I think this is partly why we get these very mixed reactions to AI where LLMs aren't quite "right" (despite often producing code that functions as well as human written code).

by strogonoff4 hours ago|

parent|

[-]

Consistency reduces the mental cost of acquiring and maintaining an understanding of a system. In a real sense, moving from one approach to two different approaches, even if one of them is slightly better than the original one, can be a downgrade.

by YZF9 hours ago|

parent|

prev|

[-]

But then you end up with nit inflation, people feel like they need to fix the nits, and do, and there's no meaning to nit any more. I try to just not comment unless I feel there is some learning from the nit.

by jiggawatts17 hours ago|

prev|

[-]

I have a crippling guilt about not keeping my apartment as spotlessly clean as my parents did theirs, to the point that I end up procrastinating, which just makes it worse.

The trick to overcoming this is not to aim for "clean" but for "cleaner than before".

Just keep chipping away at it, whether it is a messy codebase or a messy kitchen.

by zoogeny16 hours ago|

parent|

[-]

I use it for cleaning all the time. Whenever I have dishes, I always give myself permission to do as little as I want knowing that one clean dish is better than nothing. Most often I end up doing them all.

The other saying I say is "completion not perfection". That helps me in yard work especially. I'm not going for the cover shot of "Better Homes and Gardens", I just need the lawn to be cut.

by Waterluvian16 hours ago|

parent|

prev|

[-]

I call it “sweeping back the desert.”

The sand blows in endlessly. You don’t aim for a pristine, sandless land. But you can’t ignore it or it takes over.

I’ll just pick up a few things and ferry them towards their “home.” Or go do a small amount of yard work. Etc.

by YZF9 hours ago|

parent|

[-]

weeding the garden is another analogy.