build(docs): fix double build with docs-lint
Metadata
Current evaluation
Merged #513 in commit 1107517, changing docs-lint to install dependencies without a full docs build and fixing #512's redundant double build.
Resolution: closed resolved
Reason: PR #513 was approved and merged as commit 1107517. It explicitly resolves #512 by avoiding the unnecessary complete Sphinx build before docs-lint.
Evaluated by: mr-cal-bot
No scores available.
Issue body
We mistakenly thought that Sphinx's linter needed a complete build.
Resolves #512.
---
- [x] I've followed the [contribution guidelines](https://github.com/canonical/starbase/blob/main/CONTRIBUTING.md).
- [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [x] I've successfully run `make lint && make test`.
- [x] I've added or updated any relevant documentation.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #513 in commit 1107517, changing docs-lint to install dependencies without a full docs build and fixing #512's redundant double build. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #513 (commit 1107517) merged to main: make docs-lint now runs `make docs-install` instead of building the docs, eliminating the redundant double build reported in #512. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #513 (1107517): docs-lint now installs documentation dependencies instead of building the docs, fixing #512's redundant double build. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a one-line fix resolving a double build issue during docs-lint by correcting a mistaken assumption about Sphinx's linter requirements. Approved by two reviewers, passed CI, and closed issue #512. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged fix for docs-lint double build issue. Corrected the assumption that Sphinx requires a full build before linting, resolving issue #512. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/starbase#512
(confidence 100%)
PR #513 explicitly states that it resolves the docs double-build issue in #512.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #512 docs: linting builds the docs twice | starbase | closed | Fixed by merged PR #513, which changed docs-lint to depend on docs-install instead of docs, eliminating the duplicate documentation build. | |
| #525 build: fix docs-install target | starbase | merged | Merged PR #525 as commit b434201, fixing docs-install so make lint keeps the correct development dependencies installed throughout linting. | |
| #6133 build: fix docs-install target | snapcraft | merged | Merged PR #6133 as commit f6f0bd918, fixing the docs-install Make target that was breaking documentation builds after dependency setup. | |
| #508 fix: quieter docs-lint in CI | starbase | merged | Merged #508, grouping docs-lint setup and execution in CI log groups to reduce verbosity; all reported checks passed. | |
| #54 fix(docs): make readthedocs build correctly | craft-platforms | merged | Merged PR #54 via commit 16406ff, correcting the Read the Docs configuration so the documentation builds successfully. | |
| #1070 build(docs)!: add dependency starter pack 1.5 | craft-application | merged | Merged PR #1070 added dependency starter pack 1.5 to fix docs-only PR builds, landing as commit 691eec7e with all CI checks passing. | |
| #5832 build(docs): remove line length check from linter | snapcraft | merged | Merged PR #5832 in commit a634d1f08, removing the documentation line-length check from the linter in a one-line configuration change. | |
| #453 build(docs): remove line length check from linter | starbase | merged | Merged PR #453 removed the documentation line-length check from the linter in commit ec0cf32. | |
| #4358 style(docs): fix linter errors for auto-generated commands | snapcraft | merged | Merged #4358 in commit 9680075fa, generating command documentation before docs linting to prevent lint-docs failures. |