search

cdump / radiacode

Library for RadiaCode-101
MIT License
101 stars 18 forks source link

Added the possibility of building and running the project on Mac (without bluetooth) #32

Closed RamonBeast closed 2 months ago

RamonBeast commented 2 months ago

Due to bluepy not working on Mac, it's not possible to install and run the project on Mac devices as it is. I modified the project so that bluepy is excluded from the build if the platform is a Mac. I've also made some changes to basic.py to allow the main example to run on Mac as well.

ckuethe commented 2 months ago

Doesn't break on my Ubuntu machine.

cdump commented 2 months ago

and codestyle errors from tests: https://github.com/cdump/radiacode/actions/runs/8958200767/job/24765854405?pr=32

cdump commented 2 months ago

Please fix ruff linter errors from failing CI test (https://github.com/cdump/radiacode/actions/runs/9030194133/job/24818308952?pr=32) and I'll merge this

RamonBeast commented 2 months ago

I somehow missed it, fixed and pushed.

cdump commented 2 months ago

Still have code-format ruff issues - you can run tests localy: https://github.com/cdump/radiacode/blob/master/.github/workflows/tests.yml#L26

RamonBeast commented 2 months ago

Last one should be the good one!