Skip to content

Conversation

@pablo-garay
Copy link
Collaborator

@pablo-garay pablo-garay commented Oct 15, 2025

Summary
The Problem:
The automated link checker reported 5 broken links, but they all work fine in a web browser.

Why They Failed:
GitHub line anchors (#L232, #L267) - Automated tools struggle to verify these
Complex text fragments (Export-Deploy links with :~:text=) - Advanced URL syntax that checkers can't validate
403 Forbidden (OpenAI docs) - Site blocks automated requests but allows browsers
Rate-limiting tools

The Fix:
Created docs/false_positives.json with all 5 links. The build workflow will now skip checking these specific links since they're known to work but fail automated validation.

File Format:
The file uses newline-delimited JSON (NDJSON) - each link is a separate JSON object on its own line, which is what the FW-CI-templates workflow expects.
The "Check for Unknown broken links" job should now pass

@pablo-garay pablo-garay requested review from a team as code owners October 15, 2025 17:26
@copy-pr-bot
Copy link

copy-pr-bot bot commented Oct 15, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@pablo-garay
Copy link
Collaborator Author

/ok to test 5b4f7ac

@agronskiy agronskiy force-pushed the docs/false_positives branch from 5b4f7ac to c248855 Compare October 20, 2025 11:23
agronskiy
agronskiy previously approved these changes Oct 20, 2025
@agronskiy agronskiy changed the title Links valid & false positives: add to list docs: Links valid & false positives: add to list Oct 20, 2025
@agronskiy agronskiy self-requested a review October 20, 2025 12:21
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.

3 participants