search

Electrostatics / APBS_Sphinx

APBS Rewrite
http://www.poissonboltzmann.org
1 stars 3 forks source link

Update sources to PEP 492 syntax #36

Closed keith923 closed 8 years ago

keith923 commented 8 years ago

We should switch to the new syntax.

keith923 commented 8 years ago

Rather than forcing you to read that entire PEP, which I still haven't, I can tell you that this should boil down to replacing @coroutine or @asyncio.coroutine with async def and yield from with await.

keith923 commented 8 years ago

We'll need to make sure that we are bootstrapping Python 3.5, as that's the minimum version for this syntax.

keith923 commented 8 years ago

Also, with the new syntax, it's unnecessary to import asyncio if you are just using coroutine.

mtartakovsky commented 8 years ago

@keith923 should I make these changes to the other plugins?

keith923 commented 8 years ago

Complete!

keith923 commented 8 years ago

Whoops -- spoke too soon. Something is wrong with this and the CI is complaining. I should have run the unit testes before committing. Sigh.

keith923 commented 8 years ago

I've stopped flagellating myself. Turns out that Travis is not using Python 3.5. Hopefully that can be fixed.

keith923 commented 8 years ago

Simple update to travis config file.