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/07/14 13:34:36 UTC

[GitHub] [airflow] potiuk commented on issue #5587: [AIRFLOW-4953] Remove unused variables from core

potiuk commented on issue #5587: [AIRFLOW-4953] Remove unused variables from core
URL: https://github.com/apache/airflow/pull/5587#issuecomment-511203378
 
 
   Sure. If that makes it easier we can do it this way, I have no problem with that. Maybe we should add a small script to remove all files from the TODO list that have already been cleaned. This way we periodically refresh the list. I think this might then become much better framework. @basph - I hope you are ok with that as well :).
   
   And BTW in https://github.com/apache/airflow/pull/4938 I've already added a way to run pylint tests selectively per file, without leaving the docker container. This might make the whole process significantly faster.  I hope today/tomorrow we merge it.

----------------------------------------------------------------
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