← Back to issue list

docs: add linter reference page

View original Github issue

Metadata

Project
snapcraft
Number
#5032
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged documentation update adding a linter reference page. Approved by two reviewers and merged despite unrelated CI store timeout failures. Changes span two files with 62 added lines.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox run -m lint`? - [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged documentation update adding a linter reference page. Approved by two reviewers and merged despite unrelated CI store timeout failures. Changes span two files with 62 added lines.
qwen3.6-35b-a3b-mtp-q6 Merged documentation update adding a linter reference page. Approved and merged by maintainers despite unrelated CI store timeout failures.
qwen3.6-35b-a3b-mtp-q6 Merged after maintainers approved despite unrelated CI store timeout failures. The change adds a new linter reference page to the documentation.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
81%
#110 docs: add style guide starflow merged Merged pull request adding a style guide to the documentation. Approved by two reviewers with 597 lines added across two files. CI checks passed for most environments.
75%
#1302 docs: rewrite some redirects craft-parts merged Merged documentation rewrite correcting several redirects to resolve recurring lint CI failures. Approved by two reviewers, passed all CI checks, and modified four files with minimal changes.
75%
#604 docs: update inspectors reference fetch-service merged Merged documentation update adding references for undocumented inspectors, reconciling docs with actual behavior, and normalizing style. Approved by one reviewer, passed CI checks, and modified 38 files.
75%
#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.
74%
#1104 docs: update links craft-parts merged Merged after two reviewers approved. Updated documentation to fix broken links. Integrated into the main branch despite some CI test failures.
74%
#180 docs: add 'About this documentation' imagecraft merged Merged after two reviewers approved. Added an About this documentation section to the project docs. Passed lint, security, and build checks despite failing spread tests on Ubuntu variants. Changes added 70 lines across two files.
74%
#1063 docs: fix a missing link rockcraft merged Merged a documentation fix adding a missing hyperlink. Approved by four reviewers with a single-line change. The PR was merged despite one failing CI check (snap-tests), as the documentation-only update was deemed low-risk.
74%
#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.
74%
#17 ci: update docs CI craft-application merged Documentation CI configuration updated and merged to resolve Sphinx linting failures. Approved by one reviewer with passing CI checks. Changes span four files with 17 additions and 5 deletions.
74%