1803510 - Make it possible to use different .isort.cfg for different directories

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, enhancement)

Description

[Marco Castelluccio [:marco]]

This will allow us to specify certain modules as first party and address issues such as https://phabricator.services.mozilla.com/D162661#5353348.

Currently it is possible to set up separate .isort.cfg files, but isort and flake8-isort pick up a different one depending on whether they are run on the repository root or on the directory containing the .isort.cfg file.

Comment 1

[Marco Castelluccio [:marco]]

Attachment: Bug 1803510 - Set up isort as a separate linter. r=#linter-reviewers

This makes it possible to use the resolve all configs option and set up
different first party modules for different directories.

The downside is that configuration options (e.g. profile = black) have to
be repeated in every .isort.cfg file, because isort never merges configuration
files.

Comment 2

[Marco Castelluccio [:marco]]

Attachment: Bug 1803510 - Add an isort linting task. r=#linter-reviewers

Depends on D163567

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/554116951cdc
Set up isort as a separate linter. r=linter-reviewers,ahal DONTBUILD
https://hg.mozilla.org/integration/autoland/rev/2999496cb626
Add an isort linting task. r=linter-reviewers,ahal DONTBUILD

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/554116951cdc
https://hg.mozilla.org/mozilla-central/rev/2999496cb626