+1 to the idea (and to the specific config as I have already bikeshed it)

You might need to skip .py files in tests/*/data (eg. swh-web iirc)

LGTM

(I was initially surprised by the mixing together of "import" lines with "from ... import" ones, but upon reflection it makes a lot of sense, because one might have to switch between the two forms, and it's silly to have to move the line back and forth between import blocks when that happens.)

You might need to skip .py files in tests/*/data (eg. swh-web iirc)

Right ! Once global isort config will be pushed to repos, I will update D3972 with specific swh-web configuration only (django import grouping, paths exclusion).

Ok so the plan to integrate isort in all repos will be the following:

1. Integrate the following diff (manually) in all repos using the change-all-repos script from @vlorentz but do not push to remote yet

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 5ab0e08..df70797 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -26,6 +26,11 @@ repos:
     language: system
     types: [python]
 
+- repo: https://github.com/PyCQA/isort
+  rev: 5.5.2
+  hooks:
+  - id: isort
+
 - repo: https://github.com/python/black
   rev: 19.10b0
   hooks:
diff --git a/pyproject.toml b/pyproject.toml
index b5413f6..206af2c 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,2 +1,12 @@
 [tool.black]
 target-version = ['py37']
+
+[tool.isort]
+multi_line_output = 3
+include_trailing_comma = true
+force_grid_wrap = 0
+use_parentheses = true
+ensure_newline_before_comments = true
+line_length = 88
+force_sort_within_sections = true
+

2. Run make check in all repos to apply imports ordering
3. Run make test in all repos to check and fix possible side effects
4. Once it is done, add a new commit for the imports ordering
5. Push to remotes

Ok so I applied the above process locally and no bad surprises when running all test suites. I will push the changes to remote then and close that task once it is done.

awesome, thanks.