style(lint): go all-in on ty
Metadata
Current evaluation
Closed without merge. The PR introduced minor lint changes but remained unreviewed with two unresolved comments and failing CI checks, indicating it was abandoned.
Suggested action: —
No scores available.
Issue body
Describe your changes.
---
- [ ] I've followed the [contribution guidelines](https://github.com/canonical/starbase/blob/main/CONTRIBUTING.md).
- [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [ ] I've successfully run `make lint && make test`.
- [ ] I've added or updated any relevant documentation.
- [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing.
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Closed without merge. The PR introduced minor lint changes but remained unreviewed with two unresolved comments and failing CI checks, indicating it was abandoned. | |
| qwen3.6-35b-a3b-mtp-q6 |
Staleness:
5
Complexity:
15
Confidence:
85
|
needs review | Updates project linting configuration to use the ty tool. PR is newly opened by a maintainer, currently awaiting review with no comments or CI results visible. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2692 style(type): add ty ignores | charmcraft | closed | Closed without merging. The PR adding ty type ignores remained unreviewed with unresolved comments and multiple failing CI checks. The change was abandoned. | |
| #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. | |
| #56 lint: stricter ruff rules | starbase | closed | Closed without merging. The pull request applying stricter ruff linting rules remained in pending review with no comments or CI checks and was abandoned. | |
| #446 style(lint): fix typing for ty update | craft-cli | merged | Merged a maintainer pull request fixing typing for the ty linter update. Approved by one reviewer, the change adjusted three files with minor style modifications. CI checks passed and the branch was successfully integrated. | |
| #1322 style: resolve linting issues | rockcraft | closed | Closed without merging. Addressed linting issues ISC004, INP001, and PTH118/120 across five files but was abandoned due to unresolved review comments and a failing source-spread-tests CI check. | |
| #239 style(lint): autoformat | starbase | merged | Merged after approval. Applied automated linting to a single file, adjusting one line. The maintainer requested the merge post-review, completed without CI checks or unresolved comments. | |
| #205 style(lint): autofix ruff 0.6.4 linting errors | craft-store | closed | Closed without merging. Abandoned due to ruff 0.6.4 ARG001 rule changes breaking the CI pipeline. No reviews were requested, checks never ran, and the PR remained inactive for 680 days. | |
| #33 feat: add linting CI workflow | craft-actions | closed | Changes to add a linting CI workflow were closed without merging. No reviews or comments were submitted, and the branch was abandoned after 28 days of inactivity. | |
| #15 feat: make the lint workflow reusable | starflow | closed | Closed without merging. The pull request to make the lint workflow reusable received no reviews, comments, or CI checks, indicating it was likely abandoned or superseded. The proposed changes were not integrated into the codebase. | |
| #6247 style: go all-in on ty | snapcraft | merged | Merged after approval by two reviewers. The change adopts the ty linter for comprehensive typo detection, resolving issue #6007. It updated 47 files, reducing 370 lines and adding 139. |