Skip to content

Adding linkcheck (and check for other build problems)#304

Merged
Mukulikaa merged 11 commits intonumpy:mainfrom
bsipocz:MAINT_adding_linkcheck
Feb 9, 2026
Merged

Adding linkcheck (and check for other build problems)#304
Mukulikaa merged 11 commits intonumpy:mainfrom
bsipocz:MAINT_adding_linkcheck

Conversation

@bsipocz
Copy link
Member

@bsipocz bsipocz commented Feb 7, 2026

This will close #156

I expect seeing broken links here which should be fixed.

@bsipocz bsipocz added infrastructure Issues relevant to infrasructure, rather than content maintenance labels Feb 7, 2026
Copy link
Collaborator

@rossbar rossbar left a comment

Choose a reason for hiding this comment

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

Great - thanks @bsipocz ! This is a definite improvement

For whatever reason the CI run borked the first time through - when I re-ran it it failed as expected (including the errors in the log). I took the liberty of pushing up fixes for those errors. Most of them were either removing or replacing dead links (see individual commits for comments).

The last one is the latex rendering error which has been present for a while. I finally went about searching how to fix it :) - the build errors are gone and the rendering now seems to be fixed (scroll down a bit from that anchor to see the rendered TeX - compare with what's currently deployed).

Circleci in particular seems a bit flaky (there have been a couple timeouts related to this PR) - but there's no evidence it's related to the --strict. My vote would be to put this in and simply keep an eye on circleci to see if we continue getting extraneous timeouts.

Thanks again @bsipocz ! I approve this but will let someone else merge so folks have a chance to look the changes I pushed up too!

@bsipocz
Copy link
Member Author

bsipocz commented Feb 8, 2026

Thanks Ross! I meant to come back to this, but coming back to a fixed up PR is even better 😀

I'll also let the others have a look, I myself are happy with your suggested changes.

@Mukulikaa Mukulikaa merged commit 7b2b1b6 into numpy:main Feb 9, 2026
9 checks passed
@Mukulikaa
Copy link
Contributor

Thanks, @bsipocz and @rossbar.

github-actions bot pushed a commit that referenced this pull request Feb 9, 2026
* Adding linkcheck (and check for other build problems)

* DOC: rm dead quansight link.

* DOC: Update link to boolean indexing.

* DOC: Rm dead link to machinelearningmaster.com

* DOC: Replace explicit link with doc xlink.

* DOC: replace dead nih blog link with arxiv.org

* DOC: Update imageio link.

* Rm dead link to raddq.com.

* DOC: rm links to cpcb (in/com).

The .com link is dead, the .in link consistently times out.

* TEX: Add column specifications to tables.

Fixes errors (and borked rendering) in static eq tutorial.

* DOC: Rm sciencedirect link (403)

---------

Co-authored-by: Ross Barnowski <rossbar@caltech.edu> 7b2b1b6
@bsipocz bsipocz deleted the MAINT_adding_linkcheck branch February 9, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Issues relevant to infrasructure, rather than content maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: add a linkchecker job

3 participants