Patchwork D1834: pylint: add a check for multiple statement on a single line

login
register
mail settings
Submitter phabricator
Date Jan. 10, 2018, 8:23 a.m.
Message ID <differential-rev-PHID-DREV-qko73sfq3rgjicbs7m7u-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/26633/
State Superseded
Headers show

Comments

phabricator - Jan. 10, 2018, 8:23 a.m.
lothiraldan created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  Now that they are all cleaned up, we can install a check so that no new
  occurrence appears.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1834

AFFECTED FILES
  tests/test-check-pylint.t

CHANGE DETAILS




To: lothiraldan, #hg-reviewers
Cc: mercurial-devel
phabricator - Jan. 14, 2018, 7:34 p.m.
indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.


  Nice cleanup. Always happy to turn on more lint checks.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D1834

To: lothiraldan, #hg-reviewers, pulkit, indygreg
Cc: indygreg, mercurial-devel

Patch

diff --git a/tests/test-check-pylint.t b/tests/test-check-pylint.t
--- a/tests/test-check-pylint.t
+++ b/tests/test-check-pylint.t
@@ -11,7 +11,7 @@ 
 
   $ touch $TESTTMP/fakerc
   $ pylint --rcfile=$TESTTMP/fakerc --disable=all \
-  >   --enable=W0102 \
+  >   --enable=W0102,C0321 \
   >   --reports=no \
   >   --ignore=thirdparty \
   >   mercurial hgdemandimport hgext hgext3rd