Skip to content

[DOC] Fix broken link in dependencies.rst#1034

Merged
fkiraly merged 3 commits intosktime:mainfrom
kabirvashisht4-glitch:fix-broken-dependency-link-docs
Apr 9, 2026
Merged

[DOC] Fix broken link in dependencies.rst#1034
fkiraly merged 3 commits intosktime:mainfrom
kabirvashisht4-glitch:fix-broken-dependency-link-docs

Conversation

@kabirvashisht4-glitch
Copy link
Copy Markdown
Contributor

@kabirvashisht4-glitch kabirvashisht4-glitch commented Apr 6, 2026

Reference Issue :
#1033

While going through the codebase I found a broken link in
docs/source/developer_guide/dependencies.rst at line 51.

The link was pointing to a file that no longer exists because
the _check_soft_dependencies utility was moved to skbase in PR #508
but the doc link was never updated.

Changed the link from:
https://github.com/sktime/skpro/blob/main/skpro/testing/utils/validation/_dependencies.py

to:
https://github.com/sktime/skbase/blob/main/skbase/utils/dependencies/_dependencies.py

Copy link
Copy Markdown
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, good spot!

@fkiraly fkiraly added documentation Documentation & tutorials module:base-framework BaseObject, registry, base framework labels Apr 9, 2026
@fkiraly fkiraly merged commit f1b48dc into sktime:main Apr 9, 2026
2 checks passed
@kabirvashisht4-glitch
Copy link
Copy Markdown
Contributor Author

@fkiraly , could you please also review #1012 PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Documentation & tutorials module:base-framework BaseObject, registry, base framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants