-
Notifications
You must be signed in to change notification settings - Fork 19
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
cylc hub: document multi-user config post jupyter server v2 upgrade #625
Conversation
ff97709
to
803e4ec
Compare
875770f
to
826f589
Compare
Exception during build worked around by #636 (though build still fails on that PR without the changes from this one I think) |
edbe4b5
to
aec21b5
Compare
(rebased) |
aec21b5
to
1bfffa6
Compare
(rebased) |
CI is failing with:
This error is coming from |
I think the problem is we're peeling back the lid on multiple failures but only one at a time. Unfortunately I didn't think to test the slides locally |
I've removed slides from the standard list of CI builds, with #640 to track re-adding this when hieroglyph is fixed. The alternative would be to revert recent Sphinx 7 fixes and roll back to Sphinx 6. Given we're not actively using the slides I think we may as well drop them for now. |
9e6cdb5
to
1dcb7f8
Compare
Sphinx version has now been rolled back to <7 for 8.2.x (still >7.1 for 8.3.x) |
Need this to work in Sphinx <7.2 (which Tim spotted in cylc/cylc-flow#5710) diff --git a/src/user-guide/task-implementation/job-scripts.rst b/src/user-guide/task-implementation/job-scripts.rst
index a6d74a26..a5ba61d8 100644
--- a/src/user-guide/task-implementation/job-scripts.rst
+++ b/src/user-guide/task-implementation/job-scripts.rst
@@ -186,5 +186,5 @@ Task messages are validated by:
.. autoclass:: cylc.flow.unicode_rules.TaskMessageValidator
- :no-index:
+ :noindex:
Aborting Job Scripts on Error |
Does that need to be cherry-picked back? |
Merged 8.2.x PR and kicked CI here. |
* Auto-spawning of other user's servers can now be configured. * Jupyter Server security policy must be defined if Jupyter Lab or any other server extension is included in the deployment. * Simplify some sections and expand on the new configurations.
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
Co-authored-by: Ronnie Dutta <61982285+MetRonnie@users.noreply.github.com>
2c9fd65
to
0eb1d11
Compare
🤞 should pass this time... |
Requirements check-list
CONTRIBUTING.md
and added my name as a Code Contributor.