[GitHub] [airflow] pgagnon commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI

2019-06-02 Thread GitBox
pgagnon commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI URL: https://github.com/apache/airflow/pull/5238#issuecomment-498043930 @BasPH I didn't realize at first that a blacklist mechanism was in place. Nevertheless I feel that we should disable `missing-docstring` for now until

[GitHub] [airflow] pgagnon commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI

2019-05-31 Thread GitBox
pgagnon commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI URL: https://github.com/apache/airflow/pull/5238#issuecomment-49974 I wish this had been introduced as a non-failing check at least at the beginning. Pylint is a notoriously prickly linter...