You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@beam.apache.org by "Kenneth Knowles (Jira)" <ji...@apache.org> on 2022/01/12 03:20:00 UTC

[jira] [Updated] (BEAM-13051) Enable all practical pylint warnings

     [ https://issues.apache.org/jira/browse/BEAM-13051?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Kenneth Knowles updated BEAM-13051:
-----------------------------------
    Status: Open  (was: Triage Needed)

> Enable all practical pylint warnings
> ------------------------------------
>
>                 Key: BEAM-13051
>                 URL: https://issues.apache.org/jira/browse/BEAM-13051
>             Project: Beam
>          Issue Type: Task
>          Components: sdk-py-core
>            Reporter: Brian Hulette
>            Assignee: Mike Hernandez
>            Priority: P2
>
> We currently have 75 pylint checks disabled globally: https://github.com/apache/beam/blob/0863d95205410567a7ab191fd0a46308a0d54bb7/sdks/python/.pylintrc#L81
> I think some of these are impractical for Beam (e.g. [super-init-not-called|https://pycodequ.al/docs/pylint-messages/w0231-super-init-not-called.html] because of BEAM-6158), but most of them are verifying good conventions that we should try to stick to. Let's enable as many of the checks as possible, such that the only disabled checks are really impractical for Beam, or are incompatible with the Beam community's coding style.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)