diff options
| author | Mike Frysinger <vapier@chromium.org> | 2014-11-19 21:42:29 -0500 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-11-20 06:19:54 +0000 |
| commit | 529d3908ecfe9e6f9cccdc436f42955e03f3731f (patch) | |
| tree | 19580211fdf26145b9980ad7f5e5fb7f1f1d350e /cbuildbot/cbuildbot_run_unittest.py | |
| parent | 1b1882e34067307c3fc315dcb2608d4dd3a6e76f (diff) | |
| download | chromite-529d3908ecfe9e6f9cccdc436f42955e03f3731f.tar.gz | |
lint: silence assigning-non-slot in a few cases
In general this warning is useful, but in a few cases in our code, the
linter can't figure out the right answer. Silence it until upstream
can fix the issue.
BUG=chromium:431514
TEST=`cros lint` no longer complains
Change-Id: Iae22d2859c093e1d2bbdd1575cf505d1da689356
Reviewed-on: https://chromium-review.googlesource.com/230784
Reviewed-by: Don Garrett <dgarrett@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Diffstat (limited to 'cbuildbot/cbuildbot_run_unittest.py')
| -rwxr-xr-x | cbuildbot/cbuildbot_run_unittest.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/cbuildbot/cbuildbot_run_unittest.py b/cbuildbot/cbuildbot_run_unittest.py index 06d4379b9..aa8c322c8 100755 --- a/cbuildbot/cbuildbot_run_unittest.py +++ b/cbuildbot/cbuildbot_run_unittest.py @@ -445,6 +445,10 @@ class RunAttributesTest(_BuilderRunTestCase): ra = self._NewRunAttributes() value = 'foobar' + # The __slots__ logic above confuses pylint. + # https://bitbucket.org/logilab/pylint/issue/380/ + # pylint: disable=assigning-non-slot + # Set/Get a regular run attribute using direct access. ra.release_tag = value self.assertEqual(value, ra.release_tag) |