Skip to content

docs: fix links flagged by the weekly link check - #4291 - #4292

Merged
d-v-b merged 2 commits into
zarr-developers:mainfrom
d-v-b:claude/zarr-python-consistency-289ee7
Aug 26, 2026
Merged

d-v-b merged 2 commits into
zarr-developers:mainfrom
d-v-b:claude/zarr-python-consistency-289ee7

Conversation

@d-v-b

@d-v-b d-v-b commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

claude wrote this at my behest! it fixes some broken links in documentation material flagged in #4291, and updates our link checker to anticipate a 403 from stackoverflow. It also updates some html link elements to use markdown instead, which plays nicer with mkdocs and then lychee.

Summary

Convert the raw-HTML prev/next nav links in the zarr-indexing guide to markdown inside <nav markdown> blocks, so mkdocs validates and rewrites them at build time and lychee resolves them as files; the rendered HTML is unchanged. Update the MDN CORS link in the zarr-http-server guide to its post-redirect URL, and exclude the bot-blocked Stack Overflow tag page from lychee.

Assisted-by: ClaudeCode:claude-fable-5

[Describe what this PR changes and why, in your own words.]

For reviewers

[What would you most value a second look at? What are you already confident in? For a refactor, say whether behavior is meant to be unchanged.]

Author attestation

  • I am a human, these are my changes, and I have reviewed and understood every change and can explain why each is correct.

TODO

  • Add unit tests and/or doctests in docstrings
  • Add docstrings and API docs for any new/modified user-facing classes and functions
  • New/modified features documented in docs/user-guide/*.md
  • Changes documented as a new file in changes/
  • GitHub Actions have all passed
  • Test coverage is 100% (Codecov passes)

d-v-b added 2 commits August 26, 2026 20:56
Convert the raw-HTML prev/next nav links in the zarr-indexing guide to
markdown inside `<nav markdown>` blocks, so mkdocs validates and rewrites
them at build time and lychee resolves them as files; the rendered HTML is
unchanged. Update the MDN CORS link in the zarr-http-server guide to its
post-redirect URL, and exclude the bot-blocked Stack Overflow tag page
from lychee.

Assisted-by: ClaudeCode:claude-fable-5
@github-actions github-actions Bot added the needs release notes Automatically applied to PRs which haven't added release notes label Aug 26, 2026
@d-v-b
d-v-b marked this pull request as ready for review August 26, 2026 19:10
@d-v-b

d-v-b commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

this is a non-behavioral change scoped to our link checker and the docs for two subpackages, so I'm going to merge this as soon as checks pass.

@d-v-b
d-v-b merged commit 32ff2b1 into zarr-developers:main Aug 26, 2026
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs release notes Automatically applied to PRs which haven't added release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant