Skip to content

Docs: exclude stackoverflow.com from lychee link check - #1701

Merged
sbryngelson merged 1 commit into
masterfrom
lychee-exclude-stackoverflow
Jul 31, 2026
Merged

Docs: exclude stackoverflow.com from lychee link check#1701
sbryngelson merged 1 commit into
masterfrom
lychee-exclude-stackoverflow

Conversation

@sbryngelson

Copy link
Copy Markdown
Member

Description

The docs link check fails on a Stack Overflow URL in docs/documentation/docker.md:

### Errors in build/install/docs/mfc/documentation/docker.html
* [403] <https://stackoverflow.com/questions/9727688/how-to-get-the-cuda-version> (at 192:180) | Rejected status code: 403 Forbidden

Stack Overflow returns 403 to automated requests. The link resolves fine in a
browser — it is not broken, it just refuses bots. This adds it to the existing
exclude list in .lychee.toml, alongside the entries already there for the
same reason (Microsoft Store, VS Code, OLCF Summit).

Scoped to the domain rather than the single question URL, since Stack Overflow
blocks bots site-wide and any future SO link would hit the same failure.

Type of change

  • Bug fix

Testing

Reproduced and verified locally with lychee 0.24.2 against a minimal HTML file
containing the failing URL:

  • without the config: [403] ... Rejected status code: 403 Forbidden — matches CI exactly
  • with the config: 0 Errors, 1 Excluded

Also confirmed in CI: the same one-line change applied to the branch of #1628
turned the previously failing Documentation workflow green
(run 30662138744).

Checklist

  • I added or updated tests for new behavior
  • I updated documentation if user-facing behavior changed

Stack Overflow returns 403 to automated requests, so the CUDA-version
link in docs/documentation/docker.md fails the docs link check even
though it resolves fine in a browser. Same class of exclusion already
present for the Microsoft Store, VS Code, and OLCF Summit pages.
Copilot AI review requested due to automatic review settings July 31, 2026 22:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the documentation link-check configuration (lychee) to avoid false CI failures caused by Stack Overflow returning 403 to automated requests.

Changes:

  • Add stackoverflow.com to the .lychee.toml URL exclusion list so lychee skips those links.

Comment thread .lychee.toml
"https://www\\.olcf\\.ornl\\.gov/summit", # Returns 503 to automated requests
"https://apps\\.microsoft\\.com/store/detail/", # Microsoft Store detail pages return 403 to automated requests
"https://code\\.visualstudio\\.com/?$", # Root page returns 403 to automated requests
"https://stackoverflow\\.com", # Returns 403 to automated requests
@sbryngelson
sbryngelson merged commit 0c9a1d4 into master Jul 31, 2026
35 checks passed
@sbryngelson
sbryngelson deleted the lychee-exclude-stackoverflow branch July 31, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants