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

Side by Side Diff: PRESUBMIT.py

Issue 1858633002: PRESUBMIT: Update PyLint blacklist (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 import json 9 import json
10 import os 10 import os
(...skipping 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
407 source_file_filter = lambda x: input_api.FilterSourceFile(x, None, black_list) 407 source_file_filter = lambda x: input_api.FilterSourceFile(x, None, black_list)
408 results.extend(_CheckApprovedFilesLintClean( 408 results.extend(_CheckApprovedFilesLintClean(
409 input_api, output_api, source_file_filter)) 409 input_api, output_api, source_file_filter))
410 results.extend(input_api.canned_checks.RunPylint(input_api, output_api, 410 results.extend(input_api.canned_checks.RunPylint(input_api, output_api,
411 black_list=(r'^.*gviz_api\.py$', 411 black_list=(r'^.*gviz_api\.py$',
412 r'^.*gaeunit\.py$', 412 r'^.*gaeunit\.py$',
413 # Embedded shell-script fakes out pylint. 413 # Embedded shell-script fakes out pylint.
414 r'^build[\\\/].*\.py$', 414 r'^build[\\\/].*\.py$',
415 r'^buildtools[\\\/].*\.py$', 415 r'^buildtools[\\\/].*\.py$',
416 r'^chromium[\\\/].*\.py$', 416 r'^chromium[\\\/].*\.py$',
417 r'^google_apis[\\\/].*\.py$', 417 r'^mojo.*[\\\/].*\.py$',
kjellander_webrtc 2016/04/04 12:40:27 These three no longer exists.
418 r'^net.*[\\\/].*\.py$',
419 r'^out.*[\\\/].*\.py$', 418 r'^out.*[\\\/].*\.py$',
420 r'^testing[\\\/].*\.py$', 419 r'^testing[\\\/].*\.py$',
421 r'^third_party[\\\/].*\.py$', 420 r'^third_party[\\\/].*\.py$',
422 r'^tools[\\\/]find_depot_tools.py$',
423 r'^tools[\\\/]clang[\\\/].*\.py$', 421 r'^tools[\\\/]clang[\\\/].*\.py$',
424 r'^tools[\\\/]generate_library_loader[\\\/].*\.py$', 422 r'^tools[\\\/]generate_library_loader[\\\/].*\.py$',
423 r'^tools[\\\/]generate_stubs[\\\/].*\.py$',
425 r'^tools[\\\/]gn[\\\/].*\.py$', 424 r'^tools[\\\/]gn[\\\/].*\.py$',
426 r'^tools[\\\/]gyp[\\\/].*\.py$', 425 r'^tools[\\\/]gyp[\\\/].*\.py$',
427 r'^tools[\\\/]isolate_driver.py$', 426 r'^tools[\\\/]isolate_driver.py$',
427 r'^tools[\\\/]mb[\\\/].*\.py$',
428 r'^tools[\\\/]protoc_wrapper[\\\/].*\.py$', 428 r'^tools[\\\/]protoc_wrapper[\\\/].*\.py$',
429 r'^tools[\\\/]python[\\\/].*\.py$', 429 r'^tools[\\\/]python[\\\/].*\.py$',
430 r'^tools[\\\/]python_charts[\\\/]data[\\\/].*\.py$', 430 r'^tools[\\\/]python_charts[\\\/]data[\\\/].*\.py$',
431 r'^tools[\\\/]refactoring[\\\/].*\.py$', 431 r'^tools[\\\/]refactoring[\\\/].*\.py$',
432 r'^tools[\\\/]swarming_client[\\\/].*\.py$', 432 r'^tools[\\\/]swarming_client[\\\/].*\.py$',
433 r'^tools[\\\/]vim[\\\/].*\.py$', 433 r'^tools[\\\/]vim[\\\/].*\.py$',
434 # TODO(phoglund): should arguably be checked. 434 # TODO(phoglund): should arguably be checked.
435 r'^tools[\\\/]valgrind-webrtc[\\\/].*\.py$', 435 r'^tools[\\\/]valgrind-webrtc[\\\/].*\.py$',
436 r'^tools[\\\/]valgrind[\\\/].*\.py$', 436 r'^tools[\\\/]valgrind[\\\/].*\.py$',
437 r'^tools[\\\/]win[\\\/].*\.py$', 437 r'^tools[\\\/]win[\\\/].*\.py$',
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
510 for builder in masters[master]: 510 for builder in masters[master]:
511 if 'presubmit' in builder: 511 if 'presubmit' in builder:
512 # Do not trigger presubmit builders, since they're likely to fail 512 # Do not trigger presubmit builders, since they're likely to fail
513 # (e.g. OWNERS checks before finished code review), and we're running 513 # (e.g. OWNERS checks before finished code review), and we're running
514 # local presubmit anyway. 514 # local presubmit anyway.
515 pass 515 pass
516 else: 516 else:
517 try_config[master][builder] = ['defaulttests'] 517 try_config[master][builder] = ['defaulttests']
518 518
519 return try_config 519 return try_config
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698