-
Notifications
You must be signed in to change notification settings - Fork 37
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
Switch to PySide6 #171
Switch to PySide6 #171
Conversation
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.
We are in business.
I tested both the docker and non-docker workflows on MacOS (I skipped my VM).
The only caveat for using Docker is setting DISPLAY=host.docker.internal:0
in the compose file but meh.
Is this a MacOS specific thing? We can add some doc instructions or something for that? |
I would say it's not worth managing documentation for getting containers on MacOS access to |
This PR switches from
PyQt5
(GPL license) toPySide6
(LGPL license, not restrictive if you dynamically link as in this case).It should also better support macOS (see https://wiki.qt.io/Qt_for_Python) -- @eholum can you check how this one works for you?