← Back to issue list

style(lint): fix pyright errors

View original Github issue

Metadata

Project
snapcraft
Number
#4485
Type
pull request
State
merged
Author
syu-w
Labels
Created
Updated
Closed

Current evaluation

Merged after approval by three reviewers. Resolved pyright linting errors across nine files (+59/-38 lines). Codecov confirmed test coverage remained stable at 89.2%.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval by three reviewers. Resolved pyright linting errors across nine files (+59/-38 lines). Codecov confirmed test coverage remained stable at 89.2%.
qwen3.6-35b-a3b-mtp-q6 Merged a maintainer's PR resolving pyright linting errors. Modified lines achieved 100% test coverage, raising overall coverage from 89.18% to 89.20%. Changes passed CI and were integrated into main.
qwen3.6-35b-a3b-mtp-q6 Merged to resolve pyright linting errors. The update fixes type checking issues while maintaining 100% test coverage on modified lines and slightly increasing overall repository coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2229 style: fix linting errors charmcraft merged Merged changes resolving linting errors from the latest ruff and pyright. Approved by two reviewers with zero unresolved comments. Updated 8 files, removing 22 lines overall.
82%
#58 lint: fix typing issues with updated pyright craft-archives merged Merged following reviewer approval and successful CI checks. Fixed typing errors introduced by an updated pyright version across seven files. Improved overall code coverage by 0.03%.
81%
#2222 style: fix linting issues charmcraft merged Merged following approval by two reviewers. Resolved linting issues across three files with minor style adjustments (+8/-9 lines). No review comments or CI checks were required.
80%
#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.
80%
#20 style: fix linting issues craft-artifacts merged Merged after passing all CI checks and receiving approval from two reviewers. The change resolves linting issues across four files with a +11/-5 line diff, updating code style compliance.
78%
#567 ci: fix linter issues craft-parts merged Merged fix for CI linter issues across four files (+14/-6 lines). Approved by one reviewer and passed all CI checks before integration.
76%
#77 lint: fix linting issues with up-to-date linters craft-store merged Merged a maintainer-submitted pull request resolving linting errors from updated linter versions. Approved by one reviewer, passed CI, and modified 11 files (+40/-24 lines).
76%
#237 style: fix pyright config craft-cli merged Merged a two-line fix to the pyright configuration, correcting ignores to apply recursively. Approved by two reviewers, passed CI, and integrated.
75%
#479 style: drop mypy, pyright craft-cli merged Merged after approval from two reviewers; replaces mypy and pyright with ty, passing all CI checks. The change removed 341 lines and added 17 across 8 files.
75%
#342 type: fix pyright possibly-unbound error rockcraft merged Merged to resolve a pyright possibly-unbound type error. Approved by two reviewers, passed CI checks, and modified one file with three additions and two deletions.
74%