Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Duplicate
-
Affects Version/s: Current Version
-
Fix Version/s: None
-
Component/s: Zzz! / None of the above / I really don't know
-
Labels:None
Description
For a year or two, the pylint hooks on gerrit weren't working, so a large portion of the code may not be following our code style.
We could use an automated code formatting tool such as "black" or "darker" to reformat that code, and possibly keep code automatically formatted in the future.
However, our project has a few peculiar conventions, such as the distinction between single and double quotes. It'd be better if we could keep them, so code formatting tools may need some special configuration for this.
Also, if we were to do code formatting when unrelated changes are being made, care must be taken to keep the diffs readable.
We could use an automated code formatting tool such as "black" or "darker" to reformat that code, and possibly keep code automatically formatted in the future.
However, our project has a few peculiar conventions, such as the distinction between single and double quotes. It'd be better if we could keep them, so code formatting tools may need some special configuration for this.
Also, if we were to do code formatting when unrelated changes are being made, care must be taken to keep the diffs readable.
Activity
- All
- Comments
- History
- Activity
- Transitions
- Commits
SIO-2399) which has been resolved so this should probably be resolved or closed