docs: linting builds the docs twice
Metadata
Current evaluation
Closed. Implemented the suggested fix by changing the docs-lint target dependency from docs to docs-install in common.mk, preventing redundant documentation builds.
Suggested action: —
No scores available.
Issue body
## Situation
Due to an oversight, the docs builds one extra time during linting. It was mistakenly believed that sphinx-lint needed a build.
## Request
Offending line:
https://github.com/canonical/starbase/blob/39eefe06225b16377ccaea711dc12b46f18d2c82/common.mk#L331
Change to:
```diff
-docs-lint: docs ##- Lint the documentation
+docs-lint: docs-install ##- Lint the documentation
```
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Closed. Implemented the suggested fix by changing the docs-lint target dependency from docs to docs-install in common.mk, preventing redundant documentation builds. | |
| qwen/qwen3.6-35b-a3b | — | — | Resolved duplicate documentation builds during linting by updating the Makefile dependency from docs to docs-install, eliminating the redundant build step. | |
| qwen/qwen3.6-35b-a3b | — | — | Fixed by changing the Makefile lint target dependency from docs to docs-install, preventing duplicate documentation builds during linting. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Resolved by updating the Makefile lint target dependency from docs to docs-install, preventing redundant documentation builds during linting. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #513 build(docs): fix double build with docs-lint | starbase | merged | 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. | |
| #525 build: fix docs-install target | starbase | merged | Merged a build fix optimizing the docs-install target by retaining dev dependencies throughout make lint execution. Approved by reviewers and passed CI checks, yielding a faster and more correct linting process. | |
| #6133 build: fix docs-install target | snapcraft | merged | Merged a fix for the docs-install Makefile target that resolved redundant dependency installation. The change removes an extra uv sync call, ensuring docs targets work after make setup. Approved by reviewers and passed CI. |