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/01/07 01:51:10 UTC

[GitHub] XD-DENG commented on a change in pull request #4436: [AIRFLOW-3631] Update flake8 and fix lint.

XD-DENG commented on a change in pull request #4436: [AIRFLOW-3631] Update flake8 and fix lint.
URL: https://github.com/apache/airflow/pull/4436#discussion_r245534288
 
 

 ##########
 File path: airflow/settings.py
 ##########
 @@ -90,7 +90,7 @@ def timing(cls, stat, dt):
 ____  /| |_  /__  ___/_  /_ __  /_  __ \_ | /| / /
 ___  ___ |  / _  /   _  __/ _  / / /_/ /_ |/ |/ /
  _/_/  |_/_/  /_/    /_/    /_/  \____/____/|__/
- """
+ """  # noqa: W605
 
 Review comment:
   The same to @feng-tao 's question: why do we need `# noqa: W605` here? I tried to run flake8 test on the original code and no issue was raised.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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