Skip to content

fix: add actual link check step to sphinx_build_link_check.sh#286

Open
NucleiAv wants to merge 1 commit intoaboutcode-org:mainfrom
NucleiAv:fix/sphinx-link-check-script-284
Open

fix: add actual link check step to sphinx_build_link_check.sh#286
NucleiAv wants to merge 1 commit intoaboutcode-org:mainfrom
NucleiAv:fix/sphinx-link-check-script-284

Conversation

@NucleiAv
Copy link

@NucleiAv NucleiAv commented Mar 6, 2026

Fixes #284

The script was named sphinx_build_link_check and commented as "Build locally, and then check links" but only ran the HTML build step. Adds the missing sphinx-build -b linkcheck command.

The script was named sphinx_build_link_check and commented as 'Build locally,
and then check links' but only ran the HTML build step. This adds the missing
`sphinx-build -b linkcheck` command to actually check links as intended.

Fixes aboutcode-org#284

Signed-off-by: newklei <magmacicada@proton.me>
@NucleiAv NucleiAv force-pushed the fix/sphinx-link-check-script-284 branch from 6cd2875 to 3af70ac Compare March 6, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs/scripts: Fix sphinx_build_link_check.sh name and comment — script does not run link check

1 participant