chromite/lib/*.py fails lint. |
|||||||||||
Issue description
dgarrett$cros lint lib/*.py
************* Module chromite.lib.auto_updater
C: 1, 0: Too many blank lines in a row: offset:10: {} (docstring-too-many-newlines)
C: 1, 0: Too many blank lines in a row: offset:30: {} (docstring-too-many-newlines)
C: 1, 0: Too many blank lines in a row: offset:50: {} (docstring-too-many-newlines)
W:949, 4: No exception type(s) specified (bare-except)
************* Module chromite.lib.timeout_util
W:261,30: Access to a protected member _MainThread of a client class (protected-access)
14:53:10: ERROR: linter found errors in 2 files
Git logs show you introduced these. Can you clean them up?
,
Sep 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/chromite/+/7381c2dc32165daee391e927c976bcf819831ec3 commit 7381c2dc32165daee391e927c976bcf819831ec3 Author: xixuan <xixuan@google.com> Date: Mon Aug 22 22:21:26 2016 chromite: clear pylint warning. BUG= chromium:639969 TEST=None Change-Id: I6bbcf3682eb7b4e1bdc0c5a5f93d6a27acd25294 Reviewed-on: https://chromium-review.googlesource.com/373999 Commit-Ready: Xixuan Wu <xixuan@chromium.org> Tested-by: Xixuan Wu <xixuan@chromium.org> Reviewed-by: Xixuan Wu <xixuan@chromium.org> [modify] https://crrev.com/7381c2dc32165daee391e927c976bcf819831ec3/lib/timeout_util.py [modify] https://crrev.com/7381c2dc32165daee391e927c976bcf819831ec3/lib/auto_updater.py
,
Sep 2 2016
,
Oct 7 2016
,
Oct 10 2016
,
Nov 19 2016
,
Jan 21 2017
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by aut...@google.com
, Aug 23 2016