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
Fix failing check_links
job
#14243
Fix failing check_links
job
#14243
Conversation
Thanks for making a pull request to jupyterlab! |
Thanks @krassowski 👍 Looks like the last run of the |
Yes, no idea why. It only shows |
And it passed after re-triggering 🤷 |
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.
The problem is still there and needs fixing as far as I can say (please see #14248). Based on what I see now Linux Tests / check_links (pull_request) fails on master because there is a dead link in YDocument https://jupyter-ydoc.readthedocs.io/en/latest/api/classes/YDocument.html string in source/extension/documents.rst at line 42.
Note that in the last commit on master CI is red because of different test (Linux (release_test, 3.11)
) and links were not tested, also note that in the commit before that links were tested and step failed. Also see PR #14205 where link test fails for the same reason.
Benchmark reportThe execution time (in milliseconds) are grouped by test file, test type and browser. The mean relative comparison is computed with 95% confidence. Results table
Changes are computed with expected as reference.
Waiting for localhost:8888 Cell memory leaksCreate a code cellMemory change: -160 kB Leak detected: NoLeaking objects:
Leaking collections: Create a markdown cellMemory change: -132 kB Leak detected: NoLeaking objects:
Leaking collections: Create a raw cellMemory change: -181 kB Leak detected: NoLeaking objects:
Leaking collections:
File editor memory leaksCreate a fileMemory change: -78.5 kB Leak detected: NoLeaking objects:
Notebook memory leaksCreate a notebookMemory change: +37.1 kB Leak detected: YesLeaking objects:
2 passing (6m)
|
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!
References
Fixes #14248
As seen in #14238 and other PRs
check_links
job fails with:Code changes
Replace https://jupyter-ydoc.readthedocs.io/en/latest/api/classes/YDocument.html with https://jupyter-ydoc.readthedocs.io/en/latest/api/classes/YDocument-1.html
User-facing changes
None
Backwards-incompatible changes
None