Remove line length checks from CI linters
Metadata
Current evaluation
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.
Suggested action: —
No scores available.
Issue body
Discussed in an in-person sync:
We still want to keep line endings to a sane length, but we can do that through editorconfig. However, sometimes keeping with the letter of that defeats the spirit of the line length improving readability. See: https://github.com/canonical/starbase/pull/203
Actions to take from this:
* Enable `max_line_length = 80` for markdown and rst files in editorconfig
* Keep (and enable where available but not used) max line lengths in any auto-formatters
* Determine at engineering sprint what our appropriate linter line length is.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed. RST line length was set to 88 and README badges were updated to use txt inclusion. CI linter line checks were not fully removed; editorconfig now handles formatting standards while linters retain partial limits. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed. Rst line length limit was set to 88. Line length checks remain in auto-formatters and editorconfig rather than CI linters. Jira ticket CRAFT-3002 created for tracking. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5832 build(docs): remove line length check from linter | snapcraft | merged | 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. | |
| #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. |