build(docs): remove line length check from linter
Metadata
Current evaluation
Merged after approval by two reviewers. Removes the line length check from the documentation linter. All required CI checks passed, with a single line modification applied.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval by two reviewers. Removes the line length check from the documentation linter. All required CI checks passed, with a single line modification applied. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a maintainer's pull request that removes the line length check from the documentation linter configuration, streamlining the build process and eliminating unnecessary formatting constraints. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged maintainer change removing the line length check from the documentation linter configuration. This update streamlines the build process by disabling strict line length validation for docs. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #453 build(docs): remove line length check from linter | starbase | merged | Merged a change removing the line length check from the documentation linter. Approved by two reviewers, passed all CI checks, and updated a single line. | |
| #2466 build(docs): remove line length check from linter | charmcraft | merged | Merged a one-line change removing the line length check from the documentation linter. Approved by two reviewers. Accepted after passing primary CI checks, despite initial failures on macOS and Ubuntu 20.04. | |
| #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. | |
| #508 fix: quieter docs-lint in CI | starbase | merged | Merged a fix grouping CI docs-lint prerequisites to reduce log verbosity. Approved by three reviewers with all checks passing. The change modifies one workflow file, adding 18 lines to streamline continuous integration output. | |
| #4358 style(docs): fix linter errors for auto-generated commands | snapcraft | merged | Merged via rebase-and-merge after two approvals and passing CI. Fixed linter errors in auto-generated documentation by ensuring docs are built prior to linting. Modified 5 files with negligible coverage impact. | |
| #6091 fix: extra comma in linters doc | snapcraft | merged | Merged a one-line documentation fix removing an extra comma from the linters guide. Approved by two reviewers, passed all CI checks, and integrated into the main branch. | |
| #207 Remove line length checks from CI linters | starbase | closed | Closed without fully removing CI linter checks. RST line length was set to 88, and readme badges were adjusted to use txt inclusion. Line length enforcement was partially shifted to editorconfig rather than fully removed. | |
| #919 docs: fix indentation | craft-application | merged | Merged documentation fix correcting an indentation error introduced in a prior review. Approved by two reviewers, passed all CI checks, and applied a minimal two-line change. | |
| #175 docs: force max line length of 80 characters | rockcraft | merged | Merged changes enforcing an 80-character maximum line length for documentation using sphinx-lint. Applied minor fixes across 13 files, passed CI, and received approval from three reviewers. | |
| #5346 docs: add 'disable a linter' | snapcraft | merged | Merged documentation update adding a linter disable section. Content was relocated to a newly migrated page as part of a broader migration. Approved by two reviewers, passed CI, with 58 additions and 46 deletions across four files. |