Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

Test on webkit #175

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Test on webkit #175

wants to merge 7 commits into from

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Oct 13, 2023

Follow-up to #167

Related to #152

  • Add webkit to the UI test matrix
  • Add webkit ref snapshot
  • Add webkit to the update workflow

@github-actions
Copy link
Contributor

lite-badge 👈 Try it on ReadTheDocs

@jtpio
Copy link
Member Author

jtpio commented Oct 13, 2023

Trying to set a higher timeout for webkit as the packages seem to be downloaded at least:

image

@jtpio
Copy link
Member Author

jtpio commented Oct 13, 2023

cc @DerThorsten who I think has a Mac and might be able to try using the xeus python kernel in Safari directly, just to double check.

@jtpio
Copy link
Member Author

jtpio commented Oct 13, 2023

Trying to set a higher timeout for webkit as the packages seem to be downloaded at least:

Adding the timeout seems to be helping (although might still be a bit flaky):

image

@jtpio
Copy link
Member Author

jtpio commented Oct 13, 2023

Ah looks like it was stuck in the last CI run: https://github.com/jupyterlite/xeus-python-kernel/actions/runs/6507197878?pr=175

image

@DerThorsten
Copy link
Collaborator

cc @DerThorsten who I think has a Mac and might be able to try using the xeus python kernel in Safari directly, just to double check.

on Version 16.5 (18615.2.9.11.4) on my M2 it works

@jtpio
Copy link
Member Author

jtpio commented Oct 13, 2023

One of the failure might actually be related to Galata, as reported in jupyterlab/jupyterlab#15252

image

@jtpio jtpio mentioned this pull request Jan 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants