3 ms·
Absolutely! I'm grateful that I don't have to worry about code formatting any more. But I remember in one of my earliest job the company used style checkers as
by max_sendfeld 3y ago
Absolutely! I'm grateful that I don't have to worry about code formatting any more. But I remember in one of my earliest job the company used style checkers as a pre-commit hook that rejected your commit if they found trailing whitespace. That was before code formatting was part of your IDE. (Especially for us front end devs who used Notepad++ rather than an IDE at the time).
And notepad++ had no easy way of showing trailing whitespace. So every commit was a dance of commit -> read rejection log -> remove trailing whitespace -> commit again.
- EZ-E 3y agoI have memories at my post grad school where any deviations from the expected code source formatting led to a -1 on the mark. On the first few projects it was not rare to see students getting marks well into two or three digits negatives.
- bluGill 3y agoThings that don't matter much. A nice consistent style is good to have, but it isn't something worth worrying about that much.
- hudell 3y agoI once worked on a project that would not even compile if a function had a param without a comment explaining the purpose of that param written with a specific format. Every comment was validated on compile time, you couldn't even comment code just to test something. Life was hell.
- yakshaving_jgt 3y agoWhat's stopping you from configuring your editor to automatically strip trailing whitespace on save?
- max_sendfeld 3y agoMostly the fact that this was 15 years ago :-)
- skirmish 3y agoCode reviewers that made me type in trailing whitespace that editor stripped: "this change is on lines you are not modifying". Still makes my blood boil many years later.