You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/10/29 12:37:16 UTC

[GitHub] [airflow] kik-kik opened a new pull request #6459: [AIRFLOW-5243](https://issues.apache.org/jira/browse/AIRFLOW-5243) config and exceptions pylint compliance

kik-kik opened a new pull request #6459: [AIRFLOW-5243](https://issues.apache.org/jira/browse/AIRFLOW-5243) config and exceptions pylint compliance
URL: https://github.com/apache/airflow/pull/6459
 
 
   [AIRFLOW-5243](https://issues.apache.org/jira/browse/AIRFLOW-5243) config and exceptions pylint compliance
   
   ### Notes:
   ```airflow/configuration.py:199:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
   airflow/configuration.py:205:4: R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
   airflow/configuration.py:213:4: R0911: Too many return statements (7/6) (too-many-return-statements)```
   - These 3 errors still remain and I'm not sure how to fix them, too many return statements would require some refactoring but I'm unsure about exactly what is happening in that function.
   - I couldn't figure out how to fix the other two
   
   ### Description
   - As part of [AIRFLOW-4364 Integrate Pylint](https://issues.apache.org/jira/browse/AIRFLOW-4364)
   
   ### Tests
   - No code changes
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services