kellerza / pysma

Async library for SMA Solar's WebConnect interface
MIT License
59 stars 51 forks source link

update to newer async/await coroutine syntax and minor fixes #42

Closed sillygoose closed 3 years ago

sillygoose commented 3 years ago

Here is an update to the more modern async/await syntax you inquired about, plus some other minor fixes suggested by pylint.

Not sure how to test the coroutine changes other than running these changes with my multi-inverter application. I see you have some test code using pytest, I'll have to look into pytest to see what it offers but in the meantime you can consider these changes.

sillygoose commented 3 years ago

This implements the suggested reversion, hope it takes care of the updates.

kellerza commented 3 years ago

Thanks @sillygoose ! 🎉