hbmqtt icon indicating copy to clipboard operation
hbmqtt copied to clipboard

WIP: refactor to comply python3.8 and python3.9

Open HerrMuellerluedenscheid opened this issue 4 years ago • 10 comments

HerrMuellerluedenscheid avatar Dec 31 '20 00:12 HerrMuellerluedenscheid

Great to see, that someone's working on a compatible version for Python versions >3.8. Any updates regarding this PR?

resingm avatar Jan 05 '21 16:01 resingm

I'm on vacation til the end of the week and will hopefully find the time to look into this next week. A few additional errors were raised within the unit test which I need to have a closer look at.

HerrMuellerluedenscheid avatar Jan 06 '21 13:01 HerrMuellerluedenscheid

Hey, just curious.. I'm eagerly waiting for this PR to be merged.. Is the PR ready to be merged?? Thanks

pavandv avatar Jan 09 '21 19:01 pavandv

It's not ready, yet. The tests do not succeed yet because of previously introduced hiccups that need to be resolved on master beforehand, I guess.

HerrMuellerluedenscheid avatar Jan 10 '21 15:01 HerrMuellerluedenscheid

Depends on #231

HerrMuellerluedenscheid avatar Jan 10 '21 21:01 HerrMuellerluedenscheid

Tests pass for python 3.6, 3.7, 3.8 and 3.9 after reverting as discussed in #231. https://travis-ci.org/github/beerfactory/hbmqtt/builds/753846602. Still, I would like to have #231 resolved to build on solid grounds.

HerrMuellerluedenscheid avatar Jan 10 '21 22:01 HerrMuellerluedenscheid

Hey @HerrMuellerluedenscheid is this PR ready to be merged? I saw your other PR's (#228, #229) and Issue (#235) & I don't think @njouanin is ready to act on those..

If this PR is ready I would like to merge this into my own Fork.. Thanks.

pavandv avatar Jan 19 '21 08:01 pavandv

Hey @HerrMuellerluedenscheid is this PR ready to be merged? I saw your other PR's (#228, #229) and Issue (#235) & I don't think @njouanin is ready to act on those..

If this PR is ready I would like to merge this into my own Fork.. Thanks.

Hey @pavandv , No, it's not ready. I mean tests pass but I reverted a commit by @shipmints in this MR just to get the CI running. Thus, this MR now additionally depends on #236 to restore what @shipmints fixed. We need feedback/approval/merge by @njouanin.

HerrMuellerluedenscheid avatar Jan 20 '21 10:01 HerrMuellerluedenscheid

Hey @HerrMuellerluedenscheid as this is not maintained anymore, what is the status on this??

pavandv avatar Mar 02 '21 12:03 pavandv

@pavandv hey,

i continued the branch there: https://github.com/Yakifo/aio-hbmqtt/pull/18

it does not contain changes compared to this branch for the actual code, just tests.

FlorianLudwig avatar Mar 06 '21 18:03 FlorianLudwig