You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2018/05/17 19:42:10 UTC

[GitHub] mistercrunch opened a new pull request #5027: Add missing dep on contextlib2

mistercrunch opened a new pull request #5027: Add missing dep on contextlib2
URL: https://github.com/apache/incubator-superset/pull/5027
 
 
   @john-bodley community contributed https://github.com/apache/incubator-superset/pull/4927 added dep in `requirements.txt` but not in `setup.py` causing Travis to succeed but the installation of pypi to fail for many.
   
   With unpinned deps, should we have an extra suite of tests not using `requirements.txt` to catch when new releases of libs break our stuff?

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

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org