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

Posted to issues@beam.apache.org by "Mike Hernandez (Jira)" <ji...@apache.org> on 2022/03/02 23:21: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 ]

Mike Hernandez updated BEAM-13051:
----------------------------------
    Fix Version/s: 2.36.0
       Resolution: Resolved
           Status: Resolved  (was: Open)

> 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
>             Fix For: 2.36.0
>
>          Time Spent: 31h
>  Remaining Estimate: 0h
>
> 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)