Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Honestly half the reason I argue for rubocop is to avoid having readability discussions with other devs when I don't have strong opinions. I've discovered that I am nonplussed by patterns that leave other devs downright emotional and it's just not worth the effort to discuss, especially when 99% of readability issues (eg naming) are absolutely undetectable to rubocop. Having a consistent style allows splitting styling and, erm, more fruitful readability discussions.

Anyway remember you're writing code, not poetry. The inclination to act as if you're writing the latter without any kind of cost/benefit analysis is a major reason I dislike working with ruby devs. If only I were actually writing for powerpoint slides rather than for concrete business ends!



> I've discovered that I am nonplussed by patterns that leave other devs downright emotional and it's just not worth the effort to discuss

One thing that annoys me (and I'm not saying that you're doing it here) is that the people who say it's not worth arguing about are usually the ones doing the arguing. Like, if it makes no tangible difference either way, then just let people do whatever. It literally doesn't matter. But that's not acceptable to the "it's not worth discussing" people, and suddenly it becomes very important to discuss consistency. And not just any old consistency. We can't just flip a coin. No, it has to be consistent with their favourites, which are correct and therefore not worth discussing.

My personal standard is that if somebody suggests a rule and nobody really opposes it then great, enable it. If there is opposition, then it stays disabled until one side can persuade the other. If they truly care about consistency then they are welcome to change sides at any time. Attempts to browbeat the opposition should result in some private feedback from their line manager. There are exceptions for things that matter, but a lot of this stuff just doesn't.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: