Skip to content

Commit

Permalink
DOC: Update Slicer style guide for python to reflect Flake8 compliance
Browse files Browse the repository at this point in the history
  • Loading branch information
jamesobutler authored and lassoan committed Jun 22, 2022
1 parent 9390be6 commit c48a846
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions Docs/developer_guide/style_guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ Line length: Preferably keep lines shorter than 80 characters. Always keep lines

#### Python

- Indentation is 2 spaces per level for consistency with the rest of the code base. This may be revisited in the future. Do not use tabs.
- Follow the Slicer repository Flake8 configuration and run `python -m pre_commit run --all-files` to confirm compliance.
- Text encoding: UTF-8 is preferred, Latin-1 is acceptable
- Comparisons:
- To singletons (e.g. None): use 'is' or 'is not', never equality operations.
Expand All @@ -35,7 +35,6 @@ Line length: Preferably keep lines shorter than 80 characters. Always keep lines
- Local apps/library specific imports
- Slicer application imports and local/module imports may be grouped independently.
- One package per line (with or without multiple function/module/class imports from the package)
- Avoid extraneous whitespaces
- Naming conventions: when [PEP 8](https://www.python.org/dev/peps/pep-0008/#package-and-module-names) and Slicer naming conventions conflict, Slicer wins.

#### C++
Expand Down

0 comments on commit c48a846

Please sign in to comment.