fix(build): false lint errors
Metadata
Current evaluation
Merged a fix for false lint errors by ignoring the .sphinx directory and splitting sphinx-lint arguments. All CI checks passed and reviewers approved the changes.
Suggested action: —
No scores available.
Issue body
- Ignore the `.sphinx` dir.
- Split `sphinx-lint` arguments into separate lines.
---
- [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 a fix for false lint errors by ignoring the .sphinx directory and splitting sphinx-lint arguments. All CI checks passed and reviewers approved the changes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for false lint errors by excluding the .sphinx directory and splitting sphinx-lint arguments across multiple lines. Changes passed all lint and test checks. |
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. | |
| #98 ci: lint docs in all directories | starbase | merged | Merged into main. Configured CI to run sphinx-lint on all rst files across the repository. Approved by reviewer, passed all checks, and merged with zero coverage impact. | |
| #130 ci: run sphinx-lint on CI too | rockcraft | merged | Integrated sphinx-lint into the CI pipeline for automated documentation linting. The change was merged after receiving approval, passing all CI checks, and modifying three files. | |
| #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. | |
| #5 style(lint): fix linting issues | starflow | merged | Merged after approval. Resolved linting issues by modifying one file with three additions and three deletions. No unresolved comments or CI checks were present. | |
| #333 fix(common.mk): skip frivolous link checks in lint-docs | starbase | merged | Merged a fix to skip erroneous Sphinx-lint link checks in lint-docs. The change disables validation for spaces before angle brackets and underscores after backticks, resolving false warnings from a literalref directive bug. Approved and passed CI. | |
| #2137 fix(build): fix build dependencies and CI | charmcraft | merged | Merged by maintainer lengau after approval from two reviewers. Resolved build dependency and CI configuration issues across three files, adding 51 lines and removing 13. | |
| #132 fix: keep fetch-depth 0 for linting | starflow | merged | Merged to resolve docs lint breakage by keeping fetch-depth 0. The failure stemmed from re-enabling sphinx-last-updated-by-git after adopting the Starter Pack. Approved by three reviewers with all CI checks passing. | |
| #1300 build(deps): Remove comment regarding setuptools warning | rockcraft | merged | Merged. Removed the setuptools warning comment and the unused sphinxcontrib-details-directive dependency to resolve lint errors. Approved by three reviewers with passing CI checks. | |
| #3747 cli: disable new pylint false positive | snapcraft | merged | Merged a change to disable a new pylint false positive in the CLI. Approved by a reviewer, passed CI and lint checks, and modified one file. The author confirmed unrelated spread failures did not impact the change. |