-
Notifications
You must be signed in to change notification settings - Fork 44
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
add ubuntu 3.11 test #1017
add ubuntu 3.11 test #1017
Conversation
@homosapien-lcy This looks ok but a duplicate of the already existing worklfow https://github.com/enthought/enable/blob/main/.github/workflows/bleeding-edge.yml . I suggest you just update the existing one in order to not duplicate the work. |
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.
Duplicate work that should be integrated in the existing bleeding-edge
workflow
testing procedure: test results: enable/enable/examples/demo/savage: enable/enable/examples/demo/enable: kiva/examples/kiva/: |
OK, add these to the bleeding edge |
I think in both enable and chaco, the NotImplementedError is causing error in many examples, I'm not sure this is specific to 3.11. Investigating |
@homosapien-lcy I've started a manual run of the CI: https://github.com/enthought/enable/actions/runs/4509498241 |
@homosapien-lcy as you'll see in the logs, this will require a litte bit of digging to make it work:
|
Yeah, I think pyface 6.3.0 or older is not compatible with version 3.11 |
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.
Looks good.
But fails - needs to remove PySide2 (which is OK to do)
Closing in favour of #1042 |
With the latest python come out, we want to test whether enable can work under python 3.11. Since 3.11 is still not available in EDM, we setup a pypi test. closes issue #1016