3 ms·
Its bad enough when tools change whitespace without my knowledge (unless this is an option I enabled), I sure don't want them changing anything else.
by jpindar 6y ago
Its bad enough when tools change whitespace without my knowledge (unless this is an option I enabled), I sure don't want them changing anything else.
- catlifeonmars 6y agoWhy is this bad? As someone who has spent a lot of time writing code with auto formatting turned on, I’m genuinely curious which parts of your workflow break when the whitespace changes out from under you.
- xondono 6y agoMost of the pain from this comes from python repos with multiple developers (on different OS) pushing code. Using indentation for control flow was a bad decision, even if most modern IDEs now have good solutions for it.
- mook 6y agoI tried Atom for ten minutes and then gave up on it because it made undesirable whitespace changes by default. Doing code formatting is fine; doing code formatting on lines from a checkout that I didn't change is bad. Yes, the file should be reformatted. But let me know before doing it, so I can make a formatting-only commit so it's easier to review when I put up the PR.
- jpindar 6y agoWhen I'm working on new code, sure, I'll enable it, but other times when working on existing code I just want to make the minimum changes necessary. For example if I'm explaining to someone why their code doesn't work when I try it and what little change would help, sending them their code with dozens of irrelevant changes in addition to the relevant one isn't going to help my case.