Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add an isort configuration #3463

Merged
merged 8 commits into from
Jul 9, 2018
Merged

Add an isort configuration #3463

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3d3e8eb
isort config
hawkowl Jun 28, 2018
5fc001d
further tweaking
hawkowl Jun 28, 2018
5731476
Merge branch 'develop' of github.com:matrix-org/synapse into hawkowl/…
hawkowl Jun 28, 2018
ea7a787
topfile
hawkowl Jun 28, 2018
5f8bb08
this should actually be 89
hawkowl Jun 28, 2018
a83696b
Merge remote-tracking branch 'origin/develop' into hawkowl/isort
hawkowl Jul 7, 2018
b72b9d0
add tests to first party
hawkowl Jul 7, 2018
07d746b
make tests be after synapse imports
hawkowl Jul 7, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Empty file added changelog.d/3463.misc
View file
Open in desktop
Empty file.
12 changes: 12 additions & 0 deletions setup.cfg
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,15 @@ max-line-length = 90
# W503 requires that binary operators be at the end, not start, of lines. Erik doesn't like it.
# E203 is contrary to PEP8.
ignore = W503,E203

[isort]
line_length = 89
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whyso?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To match our flake8 configuration, basically.

not_skip = __init__.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whyso?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default, isort skips init files. We put a lot of implementation code in init files, so we have to run them on it too.

sections=FUTURE,STDLIB,COMPAT,THIRDPARTY,TWISTED,FIRSTPARTY,TESTS,LOCALFOLDER
default_section=THIRDPARTY
known_first_party = synapse
known_tests=tests
known_compat = mock,six
known_twisted=twisted,OpenSSL
multi_line_output=3
include_trailing_comma=true