← Back to issue list

fix(errors): remove html suffix from plugin docs slug

View original Github issue

Metadata

Project
craft-parts
Number
#1063
Type
pull request
State
closed
Author
a-velasco
Labels
—
Created
Updated
Closed

Current evaluation

PR #1063 proposed dropping .html from the plugin doc_slug. Maintainer medubelko traced the 404s to an RTD leaf-level rendering regression and fixed it via merged PR #1064; closed as superseded. Landed later as #1075 (commit 24238ae8).

Resolution: closed superseded

Reason: Maintainer @medubelko stated the .html links were a regression from a recent Read the Docs update, and patched it himself in PR #1064, which was merged; he closed #1063 to be deployed. The extensionless-URL change ultimately landed via merged PR #1075 (commit 24238ae8).

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ----- ## Issue When a package fails to build, it returns an error containing a documentation link ending in `/reference/plugins.html`. This returns a 404 error. ## Solution Remove `.html` from the `doc_slug` variable.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed superseded PR #1063 proposed dropping .html from the plugin doc_slug. Maintainer medubelko traced the 404s to an RTD leaf-level rendering regression and fixed it via merged PR #1064; closed as superseded. Landed later as #1075 (commit 24238ae8).
stealth/space-bunny-alpha — closed superseded Closed unmerged as an unnecessary workaround; the RTD rendering regression was addressed by merged PR #1064 (845bd2fa), switching documentation to extensionless URLs.
qwen/qwen3.6-35b-a3b — — Superseded and closed. The proposed slug fix was unnecessary because the root cause was an RTD regression causing 404s. The issue was resolved via a separate patch in PR #1064.
qwen3.6-35b-a3b-mtp-q6 — — Superseded. A maintainer identified an RTD update regression as the root cause and resolved it in craft-parts#1064. This pull request was closed pending deployment of the external fix.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/craft-parts#1064 (confidence 95%)

    Maintainer's replacement PR switching docs to extensionless URLs, cited in the closing comment and merged.

  • Superseded By: canonical/craft-parts#1075 (confidence 85%)

    Merged PR that removed the .html extension from doc slug (commit 24238ae8).

Related issues

Issue Project State Summary Similarity
#1075 fix: remove .html extension from doc slug craft-parts merged Merged PR #1075 in commit 24238ae8, removing the .html extension from documentation slugs to use extensionless Read the Docs URLs.
73%