fix(build): collapse docs-install logs under parent targets
Metadata
Current evaluation
Merged a Makefile update that collapses docs-install log output under parent targets to prevent CI log clutter. Approved by reviewers, passed all CI checks, and resolves the logging issue.
Suggested action: —
No scores available.
Issue body
Fixes #544
This PR changes the Makefile/common.mk structure to group the output of the dependent target `docs-install` inside the log group of its calling target (such as `docs-lint`, `docs`, or `docs-%`). This prevents the large install/setup output of documentation packages from being printed as a separate uncollapsible/cluttering log group in CI.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a Makefile update that collapses docs-install log output under parent targets to prevent CI log clutter. Approved by reviewers, passed all CI checks, and resolves the logging issue. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates to Makefile/common.mk that collapse docs-install logs under parent targets like docs-lint and docs. Fixes #544 by preventing verbose installation output from cluttering CI logs. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #594 build: run docs-clean as part of clean target | starbase | merged | Merged a one-line Makefile update adding docs-clean to the clean target. This resolves CI lint failures from stale documentation styles by ensuring make clean removes generated artifacts. Approved by two reviewers and passed all CI checks. | |
| #566 build(docs): override docs-clean target | starbase | merged | Merged. Overrides the docs-clean Makefile target to prevent accidental virtual environment deletion and resolve ambiguous test command failures during documentation cleanup. Approved by reviewers and passed all CI checks. | |
| #367 fix(build): install system packages when running `make setup` | craft-cli | merged | Merged a build fix installing system packages during make setup. Approved by two reviewers with all CI checks passing. The single-file change (+19/-10 lines) ensures build dependencies are correctly provisioned. | |
| #2280 docs: resolve local build errors | charmcraft | merged | Merged. Resolved local documentation build failures by adding files causing undefined substitution and toctree warnings to the exclude_patterns list. Approved by two reviewers and passed CI. |