mirror of
https://github.com/servo/servo.git
synced 2025-07-23 15:23:42 +01:00
Auto merge of #29920 - sagudev:py-fixups, r=mrobinson
Standalone Flake8 config Instead of passing flake8 config via CLI flags, I extracted them in flake8 config, so IDE tools can also pick it up. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [x] `./mach build -d` does not report any errors - [x] `./mach test-tidy` does not report any errors - [ ] These changes fix #___ (GitHub issue number if applicable) <!-- Either: --> - [ ] There are tests for these changes OR - [x] These changes do not require tests because it is so trivial <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
This commit is contained in:
commit
19eeca7b3a
2 changed files with 6 additions and 7 deletions
|
@ -325,15 +325,9 @@ def check_flake8(file_name, contents):
|
|||
if not file_name.endswith(".py"):
|
||||
return
|
||||
|
||||
ignore = {
|
||||
"W291", # trailing whitespace; the standard tidy process will enforce no trailing whitespace
|
||||
"W503", # linebreak before binary operator; replaced by W504 - linebreak after binary operator
|
||||
"E501", # 80 character line length; the standard tidy process will enforce line length
|
||||
}
|
||||
|
||||
output = ""
|
||||
try:
|
||||
args = ["flake8", "--ignore=" + ",".join(ignore), file_name]
|
||||
args = ["flake8", file_name]
|
||||
subprocess.check_output(args, universal_newlines=True)
|
||||
except subprocess.CalledProcessError as e:
|
||||
output = e.output
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue