Repository navigation
A cross-reference on a member's page leads to its target - #55
Merged
Merged
Conversation
Yadg.NET writes a member's page at Namespace/Type/Member.html, two folders below the root of the docs, and wrote each cross-reference in its description with one "../" in front, as on a type's page one folder up. So every link from a member's page to another type or member led into the member's own namespace folder, where there is no such page: from MultivariatePolynomial.DivideExact, MaxDegree was docs/AngouriMath.Functions/AngouriMath.Functions/MultivariatePolynomial/MaxDegree.html. That was 930 of the 1,174 broken links a check of a local build finds. Yadg.NET has not changed since July 2022 and this repository cannot push to it, so the fix is a patch the site carries, Yadg.NET.patch, which init applies to the commit it was written against. The clone keeps LF line endings, and .gitattributes keeps the patch's, so it applies on Windows too. A clone made before this keeps the old generator; uninit and init again to pick up the patch. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012sonx8iAspMiwRwokT1Ura
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.
942 of the 1,180 broken links on the API docs pages come from one line in Yadg.NET, the generator behind
/docs.A member's page is
Namespace/Type/Member.html, two folders below the root of the docs. Yadg.NET wrote each cross-reference in its description with a single../in front, as it does on a type's page one folder up. So every link from a member's page led into the member's own namespace folder, where no such page exists. FromMultivariatePolynomial.DivideExact, the link toMaxDegreewent todocs/AngouriMath.Functions/AngouriMath.Functions/MultivariatePolynomial/MaxDegree.html.Where the fix lives. Yadg.NET has not changed since July 2022, and this repository cannot push to it. So the site carries the fix as
src/Yadg.NET.patch, andinitapplies it to the commit it was written against (a79a053).core.autocrlf=false, and.gitattributeskeeps the patch LF, so it applies the same way on the Windows runner.uninitand theninitto pick up the patch.Checked:
initrun from scratch clones Yadg.NET, checks outa79a053and applies the patch.The 238 that are left:
#in them: constructors (#ctor.html) and explicit interface members. Jekyll does not serve a file whose name starts with#, and the links do not escape it. The next PRFigures 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