Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Unified Diff: tools-webrtc/gtest-parallel-wrapper.py

Issue 2812273002: Fix lint errors to enable stricter PyLint rules (Closed)
Patch Set: Rebased Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools-webrtc/get_landmines.py ('k') | tools-webrtc/ios/generate_licenses.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools-webrtc/gtest-parallel-wrapper.py
diff --git a/tools-webrtc/gtest-parallel-wrapper.py b/tools-webrtc/gtest-parallel-wrapper.py
index cb83f1907d676cfa259d0162913352d329d20537..bfdef096d5b7740e9539028ffb812a0d6d4d8696 100755
--- a/tools-webrtc/gtest-parallel-wrapper.py
+++ b/tools-webrtc/gtest-parallel-wrapper.py
@@ -8,6 +8,7 @@
# in the file PATENTS. All contributing project authors may
# be found in the AUTHORS file in the root of the source tree.
+# pylint: disable=invalid-name
"""
This script acts as an interface between the Chromium infrastructure and
gtest-parallel, renaming options and translating environment variables into
@@ -43,7 +44,7 @@ import subprocess
import sys
-def cat_files(file_list, output_file):
+def CatFiles(file_list, output_file):
with open(output_file, 'w') as output_file:
for filename in file_list:
with open(filename) as input_file:
@@ -127,7 +128,7 @@ def main():
continue
logs = [os.path.join(logs_dir, log) for log in os.listdir(logs_dir)]
log_file = os.path.join(options.output_dir, '%s-tests.log' % test_status)
- cat_files(logs, log_file)
+ CatFiles(logs, log_file)
os.rmdir(logs_dir)
return exit_code
« no previous file with comments | « tools-webrtc/get_landmines.py ('k') | tools-webrtc/ios/generate_licenses.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698