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

Run Python tests on MacOS with Python 12, replace canvas with jest-canvas-mock #16314

Merged
merged 6 commits into from May 15, 2024

Conversation

krassowski
Copy link
Member

@krassowski krassowski commented May 10, 2024

References

Code changes

None

User-facing changes

None

Backwards-incompatible changes

None

Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski krassowski changed the title Run some tests on MacOS with Python 12 Run Python tests on MacOS with Python 12, replace canvas with jest-canvas-mock May 10, 2024
@krassowski
Copy link
Member Author

It looks like vega5 needs it to. Should we just add it to @jupyterlab/testing package or is there value in only having this in individual packages?

@krassowski krassowski marked this pull request as ready for review May 10, 2024 20:05
@krassowski
Copy link
Member Author

@ericsnekbytes you may want to try out this branch - it is passing on CI already

@ericsnekbytes
Copy link
Contributor

@krassowski Just built this locally and it worked :) Thanks for your work on this!

@krassowski
Copy link
Member Author

Great! Can you tick the "Approve" on GitHub review just so that it does not show up as unreviewed later?

@ericsnekbytes ericsnekbytes self-requested a review May 14, 2024 15:11
Copy link
Contributor

@ericsnekbytes ericsnekbytes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully tested this locally.

@krassowski krassowski merged commit 2d7b0ca into jupyterlab:main May 15, 2024
82 checks passed
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.

Error during build steps on macos
2 participants