Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decode CPU temperature for Python 3 support #18

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

LionsPhil
Copy link
Contributor

This ports the changes from pimoroni/enviroplus-python#23 , where this same function is copied.

@LionsPhil
Copy link
Contributor Author

Looks like the test failure is that Python 3.4 is missing in that environment? Not sure why the rest were cancelled, unless a maintainer did it.

I've manually checked this with 3.7.3, although I haven't done any regressions for 2.x, largely because this is copying an existing fix from another Pimoroni repo with BME280 code.

@Gadgetoid
Copy link
Member

Tests auto-cancel when something explodes, looks like I've fixed them in the interim. If you have the time/inclination then a rebased PR would be welcome. Thank you.

This ports the changes from pimoroni/enviroplus-python#23 , where this same function is copied.
@LionsPhil
Copy link
Contributor Author

Been a while since I've done GitHub flow, but believe that's got you a clean rebase onto current head.

@Gadgetoid
Copy link
Member

That seems to have done the trick, thank you!

@coveralls
Copy link

Coverage Status

Coverage remained the same at 96.429% when pulling 3624f33 on LionsPhil:patch-1 into 444d3b1 on pimoroni:master.

@Gadgetoid Gadgetoid merged commit 3640070 into pimoroni:master Feb 10, 2022
@LionsPhil LionsPhil deleted the patch-1 branch February 16, 2022 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants