Skip to content

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Sep 8, 2017

Hopefully this will work around the errors we saw recently in #970 for the appveyor builds.

@pep8speaks
Copy link

pep8speaks commented Sep 8, 2017

Hello @bsipocz! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on September 09, 2017 at 00:43 Hours UTC

@astropy-bot
Copy link

astropy-bot bot commented Sep 8, 2017

Hi there @bsipocz 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog and making sure that this pull request is milestoned and labelled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

@bsipocz bsipocz force-pushed the pyopenssl_deprecation_workaround branch 2 times, most recently from c9886e4 to f22587c Compare September 8, 2017 23:28
@bsipocz bsipocz force-pushed the pyopenssl_deprecation_workaround branch from f22587c to c944476 Compare September 9, 2017 00:43
@bsipocz bsipocz merged commit c59e162 into astropy:master Sep 9, 2017
@bsipocz
Copy link
Member Author

bsipocz commented Sep 9, 2017

@aphearin - this workaround seems to do the job if you ever run into the openssl (or any other deprecation related issue again)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants