build(deps): remove unused docs dependencies
Metadata
Current evaluation
Merged after approval and CI checks. Unused documentation dependencies were removed and kept commented out pending Starter Pack removal. A reviewer requested clarification, addressed via a follow-up commit adding explanatory comments.
Suggested action: —
No scores available.
Issue body
Keep them commented out so we know which ones to remove when the Starter Pack also removes them.
---
- [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`.
~~- [ ] I've added or updated any relevant documentation.~~
~~- [ ] I've updated the relevant release notes.~~
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and CI checks. Unused documentation dependencies were removed and kept commented out pending Starter Pack removal. A reviewer requested clarification, addressed via a follow-up commit adding explanatory comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after addressing reviewer feedback. Unused documentation dependencies were removed and kept commented until the Starter Pack cleanup. A brief explanation for the exclusions was added prior to merge. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1070 build(docs)!: add dependency starter pack 1.5 | craft-application | merged | Merged after three approvals and passing all CI checks. Adds dependency starter pack 1.5 to fix documentation-only pull request build failures, updating 44 files with 1823 additions and 337 deletions. | |
| #6037 build(docs): refresh starter pack implementation | snapcraft | merged | Merged. Refreshed the documentation starter pack across 73 files, re-integrating the latest version. Kept templates for GA enablement per feedback. Passed CI, approved by three reviewers, and fixes #5071. | |
| #478 build(docs): update Starter Pack and add passthrough to docs Makefile | starbase | merged | Merged after updating the Starter Pack and adding docs Makefile passthrough targets. Resolved CI warnings by removing sphinx-toolbox, fixed make docs-help failures, and incorporated reviewer feedback. Successfully integrated. | |
| #523 docs: update Starter Pack variables | starbase | merged | Merged documentation update correcting Starter Pack variables previously missed in PR #499. Scope limited to variable name fixes only, deferring the autobuild issue to a separate upstream fix. Approved by two reviewers and passed CI checks. | |
| #1493 build(deps): update dependencies | charmcraft | merged | Merged pull request updating project dependencies. Approved by two reviewers, the maintainer addressed comments on unused packages and clarified that CI failures resulted from an external issue. | |
| #1112 build(docs): refresh starter pack implementation | rockcraft | merged | Merged an external contribution refreshing the documentation starter pack. The update integrates the latest upstream pack, adds a project-specific wordlist, and documents new make commands for building and linting. Approved by two reviewers and passed CI checks. | |
| #732 build(deps): update doc requirements | rockcraft | closed | Documentation requirements were updated separately, superseding this approved pull request. The changes passed CI and received reviewer approval before being closed without merging. |