Repository navigation
A cross-reference to something with no page is not a dead link - #57
Merged
Merged
Conversation
Yadg.NET turns a cross-reference into a link by its name alone, so it linked to pages it never wrote: 118 links on a local build. Most were types from other assemblies, PeterO.Numbers' EDecimal, EInteger and ERational and GenericTensor's GenTensor; the rest were members with no doc entry of their own, the ANTLR parser's rules and the parameters of positional records, such as Size.Count and Interval.Left. It already wrote a System type as text for the same reason. The generator now knows which pages it writes. A cross-reference to a member without a page links to its type's page, which is where a record's parameters are described, and one with no page at all is written as text. A link check of a local build goes from 121 broken links to 3, and those three come from doc comments in the library that sit on the wrong member. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura
Member
Author
|
A correction to the figures in this PR, #55 and #56, now edited in each. They came from a local build whose Rebuilt with no earlier output, at each stage:
The deployed site, checked the same way, has 5,028 pages with no broken link inside it and none to the outside. The doc comments in question are still wrong, just not in a way that breaks a link: 23 sit on the wrong member. They are fixed in the library separately. |
Rafael-SOWNet
added a commit
that referenced
this pull request
Oct 27, 2026
The generator writes every page to .output/pre and then copies all of pre into .output/final, and clean deleted only final. So a page an older build had written to pre came back on the next build, after a clean: a local build of the docs had 135 pages for members the library no longer has, such as Absf.Latexize, which is LatexizeNode now, and InternalAMExtensions.TruncateToPeriodicInterval, removed in the library's #1347. Two of them held broken links, which were then taken for the library's (corrected on #57). clean now deletes all of .output. CI builds from a fresh checkout and was never affected; this is about a build on a machine that has built before. Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Yadg.NET turns a cross-reference into a link from the referenced name alone, so it linked to pages it never wrote. On a local build that is 121 broken links:
EDecimal,EIntegerandERational, and GenericTensor'sGenTensor.Size.CountandInterval.Left.Yadg.NET already wrote a
Systemtype as plain text, for this same reason.The generator now records which pages it writes:
Checked on a local build:
Size, "with Count" now links toSize.html.None are left. The deployed site, checked the same way, has no broken link inside it and none to the outside.
Figures corrected after merging. The first version counted a local build whose
.output/prestill held pages from older builds:amsite.fsx cleanremoves only.output/final, and every page inpreis copied intofinal. The figures above come from builds with no earlier output, at each stage of the patch. The deployed site has the same 5,028 pages.🤖 Generated with Claude Code
https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura