Skip to content

MAINT - adding explicit linkcheck command#1615

Merged
rcap107 merged 4 commits into
skrub-data:mainfrom
rcap107:doc-adding-explicit-linktest
Sep 23, 2025
Merged

MAINT - adding explicit linkcheck command#1615
rcap107 merged 4 commits into
skrub-data:mainfrom
rcap107:doc-adding-explicit-linktest

Conversation

@rcap107

@rcap107 rcap107 commented Sep 17, 2025

Copy link
Copy Markdown
Member

The linkcheck function is useful for testing links that may be broken, but calling make linkcheck runs all the examples which takes forever for no good reason, so I added a noplot version to both the makefile and pyproject

@rcap107 rcap107 changed the title DOC - adding explicit linkcheck command MAINT - adding explicit linkcheck command Sep 17, 2025
@rcap107 rcap107 added the documentation Add or improve the documentation label Sep 17, 2025
@rcap107 rcap107 marked this pull request as ready for review September 17, 2025 14:55

@jeromedockes jeromedockes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

super useful, thanks!

there seem to be a couple of broken links indeed :/

should this be mentioned in the developer docs to help people know it's there?

@rcap107

rcap107 commented Sep 22, 2025

Copy link
Copy Markdown
Member Author

super useful, thanks!

there seem to be a couple of broken links indeed :/

should this be mentioned in the developer docs to help people know it's there?

I'll add a note on that and merge

@rcap107 rcap107 merged commit 96a584b into skrub-data:main Sep 23, 2025
26 checks passed
@rcap107 rcap107 deleted the doc-adding-explicit-linktest branch September 23, 2025 07:52
rcap107 added a commit to rcap107/skrub that referenced this pull request Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Add or improve the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants