You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildstream.apache.org by ro...@apache.org on 2020/12/29 13:30:13 UTC

[buildstream] 02/08: .pylintrc: Disable formatting messages

This is an automated email from the ASF dual-hosted git repository.

root pushed a commit to branch frazer/flake8
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 2dd7329540129b3d40835191170913a71b92c3df
Author: Chandan Singh <cs...@bloomberg.net>
AuthorDate: Mon Nov 11 19:02:55 2019 +0000

    .pylintrc: Disable formatting messages
    
    Since we use Black to format our code, disable code-formatting messages
    from Pylint as they are already taken care of by Black. This ensures
    that we don't get double warnings for the same problem, and that Pylint
    and Black don't fight with each other over style.
---
 .pylintrc | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/.pylintrc b/.pylintrc
index 6499644..4224064 100644
--- a/.pylintrc
+++ b/.pylintrc
@@ -64,7 +64,7 @@ confidence=
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 
-# We have two groups of disabled messages:
+# We have three groups of disabled messages:
 #
 # 1) Messages that are of no use to us
 #    This is either because we don't follow the convention
@@ -76,6 +76,10 @@ confidence=
 #    some issues that just grew out of control. Resolving these would
 #    be nice, but too much work atm.
 #
+# 3) Messages related to code formatting
+#    Since we use Black to format code automatically, there's no need for
+#    pylint to also check for those things.
+#
 
 disable=#####################################
         # Messages that are of no use to us #
@@ -111,6 +115,14 @@ disable=#####################################
 
         unused-argument,
 
+        ##################################################
+        # Formatting-related messages, enforced by Black #
+        ##################################################
+
+        bad-continuation,
+        line-too-long,
+        superfluous-parens,
+
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option