← Back to issue list

style(type): add ty ignores

View original Github issue

Metadata

Project
charmcraft
Number
#2692
Type
pull request
State
open
Author
lengau
Labels
Created
2026-05-14 14:14:32+00:00
Updated
2026-05-20 16:53:50+00:00
Closed

Current evaluation

Adds ty type-checking ignore directives, but unused suppressions in test files require removal before merging.

Suggested action: needs review

Reason: The PR is a straightforward type-checking fix from a maintainer with passing CI, but it lacks maintainer review and has been inactive for ~3.5 weeks. A maintainer should review the added ignores and address the bot warnings about unused directives before merging.

Staleness: 25 Complexity: 10 Confidence: 85

Issue body

This is just the output of `ty check --add-ignore` --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [x] I've added or updated any relevant documentation. - [x] 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. - [x] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
2026-06-15 11:11:34.304456+00:00 qwen3.6-35b-a3b-mtp-q6
Staleness: 25
Complexity: 10
Confidence: 85
needs review Adds ty type-checking ignore directives, but unused suppressions in test files require removal before merging.
2026-06-15 11:09:20.719214+00:00 pending