Date: 2019-01-14
Accepted
Inconsistent code formatting slows down development and the review process.
Code should be linted for things like:
- unused imports and variables
- consistent import order
Code formatting should be done automatically or programmatically, taking the burden off of reviewers.
For linting, use flake8 and flake8-import-order.
For formatting, use black.
All linting and formatting are enforced programmatically.
Most linting and formatting errors can be autofixed.
Text editors and IDEs are able to integrate with our linting and formattings tools to automatically fix (most) errors on save.