style: add TC001, TC002 and TC003 rules
Metadata
Current evaluation
Merged. Added TC001, TC002, and TC003 linting rules. Approved by two reviewers, passed all CI checks, and modified one file with three additions and three deletions.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Added TC001, TC002, and TC003 linting rules. Approved by two reviewers, passed all CI checks, and modified one file with three additions and three deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. The pull request successfully added TC001, TC002, and TC003 style rules after passing lint and test checks. Changes were integrated into the codebase by the maintainer. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #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. | |
| #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. | |
| #528 style(lint): ignore FBT001 and FBT002 in tests | starbase | merged | Merged. Updates lint configuration to ignore FBT001 and FBT002 rules in test files, as boolean arguments are standard testing patterns. Approved by two reviewers, passed CI checks, and applied with a minimal two-line change to align with the ecosystem. | |
| #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. | |
| #797 perf: add TC001/2/3 linter rules to check imports | craft-application | merged | Merged after approval. Introduces TC001/2/3 linter rules to validate imports and resolves associated errors across 29 files. Core CI checks passed. Implements gradual improvements to import handling precision. | |
| #242 style: fix ty linting error | craft-platforms | merged | Merged a single-file change fixing a ty linting error. Approved by one reviewer and passed CI checks. The one-line modification resolves the style violation. |