← Back to issue list

extractors: address errors from mypy uprev

View original Github issue

Metadata

Project
snapcraft
Number
#2811
Type
pull request
State
merged
Author
cjp256
Labels
Created
Updated
Closed

Current evaluation

Merged after resolving mypy type-checking errors from an upstream version update. The change modified three files (+28/-22 lines) and received single reviewer approval with zero unresolved comments.

Suggested action:

No scores available.

Issue body

- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after resolving mypy type-checking errors from an upstream version update. The change modified three files (+28/-22 lines) and received single reviewer approval with zero unresolved comments.
qwen3.6-35b-a3b-mtp-q6 Merged changes fixing type-checking errors in the extractors module following a mypy version upgrade.
qwen3.6-35b-a3b-mtp-q6 Merged changes resolving type checking errors introduced by a mypy version upgrade. The update aligns the extractors module with stricter static analysis requirements, ensuring all linting checks pass.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2807 meta: address errors from mypy uprev snapcraft merged Merged following two reviewer approvals and passing CI. Resolves mypy type-checking errors from a version upgrade. Changes are mostly non-runtime affecting, with minor improvements to edge-case handling across six files.
83%
#2816 assorted: address errors from mypy uprev snapcraft merged Merged a pull request addressing mypy errors from an upgrade. The maintainer combined remaining fixes into one PR, which was approved by two reviewers and passed CI. Changes affect 15 files with 65 additions and 45 deletions.
80%
#2812 store: address errors from mypy uprev snapcraft merged Merged after approval and passing CI. Resolves mypy type-checking errors from an upstream version update, modifying 7 files with 47 additions and 19 deletions.
79%
#2805 plugins: address type errors for mypy uprev snapcraft merged Merged type fixes for plugins to resolve mypy errors from an upstream update. Changes do not affect runtime behavior. Approved by two reviewers and modified three files with eight additions and five deletions.
79%
#2813 pluginhandler: address errors from mypy uprev snapcraft merged Merged after resolving mypy type checking errors in the pluginhandler module following a version upgrade. Approved by one reviewer, passed CI, and updated one file with 13 additions and 11 deletions.
75%
#2809 project-loader: address errors from mypy uprev snapcraft merged Merged to address mypy upgrade errors in the project-loader module. Approved by one reviewer, the changes modify 9 files with 62 additions and 53 deletions, successfully integrating the type-checking fixes.
74%
#2810 build-providers: address errors from mypy uprev snapcraft merged Merged after approval. Addresses type-checking errors in the build-providers module caused by a mypy version upgrade. The changes modify six files, adding 117 lines and removing 48.
73%
#856 style: update mypy and fix linting issues craft-application merged Merged after receiving approval from two reviewers and passing all CI checks. The update refreshed mypy and resolved linting issues across two files, resulting in 38 additions and 32 deletions.
72%
#3001 repo: type annotations and mypy fixes for base snapcraft merged Merged after approval, adding type annotations and mypy fixes to the base repository module. Single reviewer approved with zero unresolved comments. Changes modified two files, adding 25 lines and removing 22.
72%
#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.
71%