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

PR: Add utility functions that convert between ints and Qt Enums (compat.py) #481

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zjp
Copy link
Contributor

@zjp zjp commented Apr 19, 2024

This PR implements the functions I asked about in #356

@zjp
Copy link
Contributor Author

zjp commented Apr 19, 2024

I can see some failures on PySide2 that I'll address when I get to a machine that has a binary PySide2 available (I usually work on an arm64 mac).

@coveralls
Copy link

coveralls commented Jul 20, 2024

Coverage Status

coverage: 90.164% (+0.1%) from 90.016%
when pulling 72e9f44 on zjp:int_as_enum
into 0f7b181 on spyder-ide:master.

@zjp zjp force-pushed the int_as_enum branch 4 times, most recently from fef4bda to 2dd8d02 Compare July 21, 2024 00:45
@zjp
Copy link
Contributor Author

zjp commented Jul 21, 2024

Sorry it's taken me so long to update this PR. It looks like the last remaining failure is in PyQt5 on Python 3.7/Windows/Conda. The version in the runner is 5.9.2, but the last version available on Conda for that platform/tool combination is 5.15.7 so it would be possible to use almost the same version as the non-Conda version (5.15.10). Let me know your thoughts.

@zjp
Copy link
Contributor Author

zjp commented Jul 21, 2024

I found a workaround.

@dalthviz dalthviz changed the title compat.py: Add utility functions that convert between ints and Qt Enums PR: Add utility functions that convert between ints and Qt Enums (compat.py) Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Is there a need for enumasint or intasenum?
3 participants