Remove excluded files from black.yml
Categories
(Developer Infrastructure :: Lint and Formatting, task)
Tracking
(firefox84 fixed)
Tracking | Status | |
---|---|---|
firefox84 | --- | fixed |
People
(Reporter: rstewart, Assigned: rstewart)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
These files were excluded from auto-formatting since black
introduces errors that conflict with other lints; they should be manually removed from this list after noqa'ing the relevant lines.
Assignee | ||
Comment 1•4 years ago
|
||
Looks like the flake8 and black linters disagree about what the maximum line length should be (black will reformat lines to be 100 characters long, but flake8 wants every line to be at most 99 characters). This introduces issues in files like taskcluster/taskgraph/transforms/bouncer_aliases.py
. Not sure if there's a configuration that needs to be updated somewhere?
Comment 2•4 years ago
|
||
what about updating the flake8 config to be 100?
It is defined here: https://searchfox.org/mozilla-central/source/.flake8#2
Assignee | ||
Comment 3•4 years ago
|
||
These files were omitted from the original patch because reformatting them required some manual intervention in order to avoid breaking unit tests. Generally the noqa
lines were already there and just needed to be moved from one line to another (due to the reformatting by black
), but sometimes black
saw fit to move a bunch of stuff all onto one line, requiring me to introduce new noqa
lines.
Besides the autoformat by black
and some manual fixups, this patch contains no other changes.
Comment 7•4 years ago
|
||
Backed out for conflicts when backing out bug 1518999.
Backout link: https://hg.mozilla.org/integration/autoland/rev/9006d6f3cb29754037aa0b5ef4c9b2ae67006459
Assignee | ||
Updated•4 years ago
|
Comment 9•4 years ago
|
||
bugherder |
Updated•2 years ago
|
Description
•