[
https://issues.apache.org/jira/browse/AIRFLOW-4757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16860100#comment-16860100
]
ASF GitHub Bot commented on AIRFLOW-4757:
-----------------------------------------
potiuk commented on pull request #5400: [AIRFLOW-4757] Selectively disable
missing docstrings for tests
URL: https://github.com/apache/airflow/pull/5400
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> 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)
