You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2019/06/10 15:52:00 UTC
[jira] [Commented] (AIRFLOW-4757) Selectively disable some pylint
checks for tests
[ https://issues.apache.org/jira/browse/AIRFLOW-4757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16860101#comment-16860101 ]
ASF subversion and git services commented on AIRFLOW-4757:
----------------------------------------------------------
Commit f710a0db493f89829849fb17230060f91e3925d2 in airflow's branch refs/heads/master from Jarek Potiuk
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=f710a0d ]
[AIRFLOW-4757] Selectively disable missing docstrings for tests (#5400)
> Selectively disable some pylint checks for tests
> ------------------------------------------------
>
> Key: AIRFLOW-4757
> URL: https://issues.apache.org/jira/browse/AIRFLOW-4757
> Project: Apache Airflow
> Issue Type: Sub-task
> Components: ci
> Affects Versions: 2.0.0
> Reporter: Jarek Potiuk
> Assignee: Jarek Potiuk
> Priority: Major
>
> Maybe it is a good idea to selectively disable missing-docstrings for tests ? This seems to be a huge overhead having to document all test modules/classes and especially methods. And it seems easy to do with the pylint_ci script we have.
>
> After a bit of discussion with [~ash] , [~kamil.bregula] and [~BasPH] we decided to exclude those checks for testS:
> * missing-docstrings
> * no-self-use
> * too-many-public-methods
> * protected-access
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)