-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Prepare for jupyter_server preferred_dir fix #324
Conversation
Codecov ReportBase: 81.05% // Head: 81.27% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #324 +/- ##
==========================================
+ Coverage 81.05% 81.27% +0.21%
==========================================
Files 22 22
Lines 1737 1752 +15
Branches 334 334
==========================================
+ Hits 1408 1424 +16
Misses 227 227
+ Partials 102 101 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
I'm not sure what the CI error is about, but I'm assuming it is unrelated? |
CI failures are related to jupyterlab/maintainer-tools#138 |
@vidartf could you confirm this is waiting for a new release of jupyter-server? |
@fcollonval This was waiting for a final structure on the jupyter-server PR. Now that it has been merged, I will update this PR. After that it can be be merged and ideally released before the jupyter-server release. |
One of the tests that call |
@vidartf does it raise an error if you return instead of raising skip? |
I just released a new version of pytest-jupyter, kicking the tests. |
Ugh, apparently |
Okay, we're good now. The lint job failed because it couldn't yet find the new pytest-jupyter, and the macos jobs are queued due to the recent jupyterlab release. |
Thanks a lot @blink1073 for quickly fixing the pytest package |
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 @vidartf
Is it okay if I merge and then release jupyterlab_server
?
jupyterlab_server 2.18.0 has been released with this feature |
Support relocated trait in jupyter_server (jupyter-server/jupyter_server#983).
Falls back on the old trait name if the new one is undefined or ill defined.