You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Sean Busbey (JIRA)" <ji...@apache.org> on 2016/02/16 16:19:18 UTC

[jira] [Created] (YETUS-309) pylint plugin false positives

Sean Busbey created YETUS-309:
---------------------------------

             Summary: pylint plugin false positives
                 Key: YETUS-309
                 URL: https://issues.apache.org/jira/browse/YETUS-309
             Project: Yetus
          Issue Type: Bug
          Components: Test Patch
    Affects Versions: 0.1.0
            Reporter: Sean Busbey


the pylint plugin is flagging some changes as failures when the pylint utility returns a success return code because of INFO level messages about local disables.

e.g. in a patch that removes all outstanding issues but has to use local disables in some cases, test-patch returns a -1 and the diff contains only:

{code}
some/path/to/script.py:53: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
some/path/to/script.py:54: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
some/path/to/script.py:55: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
some/path/to/script.py:56: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
some/path/to/script.py:57: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
some/path/to/script.py:58: [I0011(locally-disabled), ] Locally disabling no-member (E1101)
{code}

In this particular case, the code uses dynamic attributes, which pylint in python 2.x can't handle. In general, things at the INFO level (IXXXX) shouldn't cause a failure. Or at a minimum personalities should be able to opt-in to it not failing without overriding the log output filter.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)