-
Notifications
You must be signed in to change notification settings - Fork 18
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
Added the possibility of building and running the project on Mac (without bluetooth) #32
Conversation
…ecting to the Radiacode via USB
Doesn't break on my Ubuntu machine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR, I have two suggestions for code style
and codestyle errors from tests: https://github.com/cdump/radiacode/actions/runs/8958200767/job/24765854405?pr=32 |
Please fix |
I somehow missed it, fixed and pushed. |
Still have code-format ruff issues - you can run tests localy: https://github.com/cdump/radiacode/blob/master/.github/workflows/tests.yml#L26 |
Last one should be the good one! |
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.