-
Notifications
You must be signed in to change notification settings - Fork 9
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
WIP: Jupyterlab v3 to v4 dependencies update #32
Conversation
fix pytest
fixing jest
adding jest-environment-jsdom because yes
adding @jupyterlab/testing
adding jest-junit
fixing path
fixing path
fixing path
fixing path
Changes to enable installing the extension
WIP: Update of jypyterlab v3 to jupytrelab v4 dependencies
Environment dependencies
pyproject.toml
Outdated
@@ -29,7 +29,7 @@ dependencies = [ | |||
"peewee", | |||
"psutil", | |||
"pyjwt", | |||
"jsonschema", | |||
"jsonschema<4.20", |
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.
@garciagenrique This restriction makes it incompatible with rucio_clients==34 that requires: jsonschema ~=4.20.0 . Is this intended?
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.
I think this was some leftovers from the tests with the different jsonschema versions. I would suggest to relax all the dependencies as much as possible and let the different developers/package managers install the needed dep versions for each rucio-jupyterlab version
Jupyterlab v3 to v4 dependencies update.
This update compiles with
jupyterlab=4.1
,python=3.11
,notebook<7
, andNodeJS=~20
TODO before merging:
v0.10.0
from current master branchCo-authored-by: Giovanni Guerrieri giovanni.guerrieri@cern.ch
Co-authored-by: Francesc Torradeflot torradeflot@pic.es