Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@scriptsrc
Copy link
Contributor

Reverting PR 667. Adding line to quickstat to fix InsecurePlatformWarning.

@kpcyrd - Thoughts?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 60.292% when pulling 46c75ed on issue_620_attempt_2 into e817b8f on develop.

@scriptsrc scriptsrc changed the title Reverting PR 667. Adding line to quickstat to fix InsecurePlatformWarning… Reverting PR 667. Adding line to quickstart to fix InsecurePlatformWarning… Apr 18, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 60.281% when pulling 97b0813 on issue_620_attempt_2 into e817b8f on develop.

@kpcyrd
Copy link

kpcyrd commented Apr 19, 2017

You might want to bundle them into one RUN statement to keep the number of layers low. :)

RUN pip install setuptools --upgrade &&\
  pip install pip --upgrade &&\
  pip install "urllib3[secure]" --upgrade

@scriptsrc
Copy link
Contributor Author

For some reason it failed on me when I bundled them.

@scriptsrc scriptsrc merged commit db03221 into develop Apr 19, 2017
@scriptsrc scriptsrc deleted the issue_620_attempt_2 branch May 25, 2017 05:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants