diff options
| author | Mike Frysinger <vapier@chromium.org> | 2014-11-12 23:16:48 -0500 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-11-13 08:52:30 +0000 |
| commit | 422faf4b4a5283cb82d64ea9d3070660db153135 (patch) | |
| tree | 0195697880730a50dea30f4c33e75ca650cd6238 /cbuildbot/cbuildbot_config_unittest.py | |
| parent | 57b87d117d98c41245f4b5e797987c7d96f65cdb (diff) | |
| download | chromite-422faf4b4a5283cb82d64ea9d3070660db153135.tar.gz | |
lint: fix superfluous parenthesis
Newer pylint catches warnings like:
C:1950, 0: Unnecessary parens after 'if' keyword (superfluous-parens)
BUG=chromium:431514
TEST=`./cbuildbot/run_tests` passes
TEST=`cros lint` no longer complains about superfluous-parens
Change-Id: I47f901f8ceb06402e3c2e83303b7be38390ae535
Reviewed-on: https://chromium-review.googlesource.com/229510
Reviewed-by: Gabe Black <gabeblack@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Diffstat (limited to 'cbuildbot/cbuildbot_config_unittest.py')
| -rwxr-xr-x | cbuildbot/cbuildbot_config_unittest.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cbuildbot/cbuildbot_config_unittest.py b/cbuildbot/cbuildbot_config_unittest.py index e362d7666..f02d32e9b 100755 --- a/cbuildbot/cbuildbot_config_unittest.py +++ b/cbuildbot/cbuildbot_config_unittest.py @@ -438,7 +438,7 @@ class CBuildBotTest(cros_test_lib.MoxTestCase): def testNoDuplicateSlavePrebuilts(self): """Test that no two same-board paladin slaves upload prebuilts.""" for cfg in cbuildbot_config.config.values(): - if (cfg['build_type'] == constants.PALADIN_TYPE and cfg['master']): + if cfg['build_type'] == constants.PALADIN_TYPE and cfg['master']: slaves = cbuildbot_config.GetSlavesForMaster(cfg) prebuilt_slaves = [s for s in slaves if s['prebuilts']] # Dictionary from board name to builder name that uploads prebuilt |