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 2018/08/04 17:53:53 UTC

[GitHub] tedmiston commented on issue #3691: [AIRFLOW-2846] Add missing python test dependency to setup.py

tedmiston commented on issue #3691: [AIRFLOW-2846] Add missing python test dependency to setup.py
URL: https://github.com/apache/incubator-airflow/pull/3691#issuecomment-410466590
 
 
   Good catch.  I'm not sure how we made it this long without `tox` in there.
   
   I happen to install it system-wide with pipsi... I wonder if everyone else just happens to do something similar, or maybe they're running the dockerized tests setup, or maybe they're just not running against tox locally?  I wonder if there's room for us to make that clearer in the docs?  I'm not sure if we consider one of these a best practice more than the others.

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