Skip to content

[config] Point doc urls at live documentation - #5007

Merged
bruntib merged 1 commit into
Ericsson:masterfrom
gamesh411:labels/6.29.0-doc-urls
Aug 7, 2026
Merged

[config] Point doc urls at live documentation#5007
bruntib merged 1 commit into
Ericsson:masterfrom
gamesh411:labels/6.29.0-doc-urls

Conversation

@gamesh411

@gamesh411 gamesh411 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Checked every doc_url in the label files and 146 were dead: upstream
rebuilds its docs from trunk, so when a checker is renamed or dropped,
the link rots silently.
Where the checker is still documented but under a changed anchor, the
url is re-anchored.
Where it is gone from trunk, the url is pinned to the last version's url
that had the docs for that checker.
The hicpp check pages are no longer generated on trunk at all, so those
are pinned as well.
There was also one mistyped host and two wrong markdownlint anchors.

Five infer entries had CodeChecker's severity token substituted into the
anchor as well as into the checker name, so they get infer's own token
back, and two more carried a stray empty fragment.
The two entries left alone point at issue types that no published
version of the infer docs describes.

Checked every doc_url in the label files and 146 were dead: upstream
rebuilds its docs from trunk, so when a checker is renamed or dropped,
the link rots silently.
Where the checker is still documented but under a changed anchor, the
url is re-anchored.
Where it is gone from trunk, the url is pinned to the last version's url
that had the docs for that checker.
The hicpp check pages are no longer generated on trunk at all, so those
are pinned as well.
There was also one mistyped host and two wrong markdownlint anchors.

Five infer entries had CodeChecker's severity token substituted into the
anchor as well as into the checker name, so they get infer's own token
back, and two more carried a stray empty fragment.
The two entries left alone point at issue types that no published
version of the infer docs describes.
@gamesh411
gamesh411 marked this pull request as ready for review August 5, 2026 16:06
@gamesh411
gamesh411 requested a review from bruntib as a code owner August 5, 2026 16:06
@gamesh411

Copy link
Copy Markdown
Contributor Author

The config coverage is fixed in #5006, so I did not implement it here. Once that PR is merged, this PR will merge in master.

@bruntib
bruntib merged commit 269e47a into Ericsson:master Aug 7, 2026
12 of 13 checks passed
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.

2 participants