After watching a talk about code review, from the last EuroPython, I
got some ideas that I'd like to summarise.
The first one is a point I made on my talk about clean code in
Python, about running style checks
(PEP-8, for example), automatically as part of the CI. This has several
rationales, the first one being that it's the sort of thing machines
should do, and not humans. Having software engineers reviewing the tabs,
spacing, and such, is just a wasted effort (not to mention, wasted
money). As I mentioned on the conclusions during my talk, we should
focus on the readability of the code in terms of the logic, by asking
ourselves things like "does this code make sense?", rather than
"is the space before the comma correct?". Remember, code is for
humans, not machines, it's a means of communication with the
development team.