-
Notifications
You must be signed in to change notification settings - Fork 39
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 broken links detected by make linkcheck-files #484
Conversation
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.
Wow! Thanks for catching all those!
I caught one small issue and left a comment
Thanks!! |
Hi all - any other suggestions here? |
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.
LGTM! Let's get this in. Thanks for the ping @melissawm
Sorry I dropped the ball on this one 😭 |
# References and relevant issues Closes #439 Closes #328 Depends on #484 # Description Adds a sphinx linkcheck step to be run once a week. The reported failures will show up as issues in the repo. This check can also be triggered manually. I am using the sphinx linkchecker as it will check all links in the docs, including those coming from docstrings. You can see this in action on my fork, here: melissawm#16
References and relevant issues
Related to
#474
Description
Fixes a number of broken links detected by
make linkcheck-files
.A few comments:
further-resources/sample_data.md:17: [redirected permanently] [digitalpathologyassociation.org/whole-slide-imaging-repository](https://digitalpathologyassociation.org/whole-slide-imaging-repository) to [https://digitalpathologyassociation.org?preview_error](https://digitalpathologyassociation.org/?preview_error)
. I tried googling but only found this: http://www.wsirepository.org/ and it doesn't load. Any ideas?developers/architecture/app_model.md:288: [broken] https://github.com/napari/napari/blob/main/napari/_app_model/_submenus.py: 404 Client Error: Not Found for url: https://github.com/napari/napari/blob/main/napari/_app_model/_submenus.py
I couldn't find SubmenuItem in the codebase, maybe I'm missing somethingtutorials/tracking/cell_tracking.md:229: [broken] https://github.com/quantumjot/btrack/blob/main/models/cell_config.json: 404 Client Error: Not Found for url: https://github.com/quantumjot/btrack/blob/main/models/cell_config.json
. Should we replace with a new link?A few items are left, mostly links to old code. I will try to replace with permalinks where I can, but not sure if I'll be able to find all of them. If that's the case I might try to ping the original authors for an appropriate substitution.