Closed Bug 1672023 Opened 3 months ago Closed 3 months ago

Remove excluded files from black.yml


(Firefox Build System :: Lint and Formatting, task)



(firefox84 fixed)

84 Branch
Tracking Status
firefox84 --- fixed


(Reporter: rstewart, Assigned: rstewart)




(1 file)

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.

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/ Not sure if there's a configuration that needs to be updated somewhere?

Flags: needinfo?(sledru)

what about updating the flake8 config to be 100?
It is defined here:

Flags: needinfo?(sledru)

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.

Pushed by
Remove excluded files from `black.yml` r=sylvestre,perftest-reviewers,geckoview-reviewers,agi
Backout by
Backed out changeset 94ec15429e21 for Backout conflicts with Bug 1654103. CLOSED TREE
Pushed by
Remove excluded files from `black.yml` r=sylvestre,perftest-reviewers,geckoview-reviewers,agi
Flags: needinfo?(rstewart)
Pushed by
Remove excluded files from `black.yml`
Flags: needinfo?(rstewart)
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.