-
Notifications
You must be signed in to change notification settings - Fork 364
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
update leaflet-splitmap to v1.0.3 #1135
update leaflet-splitmap to v1.0.3 #1135
Conversation
Neat!! I guess the CI failure is very much unrelated. I'm surprised there is no change in the package-lock/yarn.lock (whichever lives on this repository) |
cfd4292
to
25848cb
Compare
added yarn.lock changes |
Thank you for fixing this. Can't wait for the split-map to work again! |
5506a80
to
c0bc542
Compare
@martinRenou 2 of 4 tasks fail. I have no spare time to check issues. anyways,
|
For the UI-tests, what you can do is try to revert @jupyterlab/builder dependency change and make sure to use JupyterLab 3 in the CI. The issue is that UI-tests (and other things) will need to be updated to work against JupyterLab 4 for the dev setup. Pinning JupyterLab 3 in the CI does not mean ipyleaflet will not work in JupyterLab 4 in userland, so we're good to pin it for now. Concerning the libmamba thingy, I feel like we should use micromamba in the CI instead of conda/mamba using this action: https://github.com/mamba-org/setup-micromamba |
56503f1
to
54a10f0
Compare
54a10f0
to
70f66bf
Compare
06c2b9a
to
d98bd4f
Compare
d98bd4f
to
1ed3c6b
Compare
@martinRenou gotcha, I set requirements to But with v3.6.5 the docs build fails as
and this does not happen for jupyterlab v4.0.5. Do you have any hints how to fix it? Dependencies are
vs
|
8b65d84
to
1ed3c6b
Compare
bbd86cd
to
f65e486
Compare
f65e486
to
1e117af
Compare
@martinRenou 3 check out of 4 pass 🎉 Additionally I had to: |
Great! Thanks! Would it help if I push a commit to try and fix the latest build? |
@martinRenou sure, that would be great! I opened #1136 on top of this PR but don't hesitate to push into this PR to make CI green |
Ok, let's merge this one as is, and see if your other PR gets everything green! Thanks a lot for your 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.
Thanks!
Fixes: #1066
Screen.Recording.2023-09-07.at.15.29.04.mov
@martinRenou fyi