Open
Bug 1675650
Opened 4 years ago
Updated 2 years ago
Make line lengths with `mach lint` consistent with flake8
Categories
(Developer Infrastructure :: Lint and Formatting, task)
Developer Infrastructure
Lint and Formatting
Tracking
(Not tracked)
NEW
People
(Reporter: rstewart, Unassigned)
References
(Blocks 1 open bug)
Details
This can be fixed with the following patch:
diff --git a/tools/lint/python/black.py b/tools/lint/python/black.py
index 8983ab33c138..9e065ee6e289 100644
--- a/tools/lint/python/black.py
+++ b/tools/lint/python/black.py
@@ -119,7 +119,7 @@ def run_black(config, paths, fix=None, *, log, virtualenv_bin_path):
log.debug("Black version {}".format(get_black_version(binary)))
- cmd_args = [binary]
+ cmd_args = [binary, "--line-length", "99"]
if not fix:
cmd_args.append("--check")
base_command = cmd_args + paths
(No reason to do it now since it'll introduce a lot of changes, but before the next time the tree is auto-formatted, this should be done as well.)
Updated•2 years ago
|
Product: Firefox Build System → Developer Infrastructure
You need to log in
before you can comment on or make changes to this bug.
Description
•