← Back to issue list

feat(Makefile): add ty as an optional linter

View original Github issue

Metadata

Project
starbase
Number
#429
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Added ty as an optional Makefile linter and merged the change in c2390f0; it remained non-default until considered production-ready.

Resolution: closed resolved

Reason: PR canonical/starbase#429 was approved with all CI checks passing and merged as commit c2390f0, adding opt-in ty linting without changing the default linter set.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This adds the ability to lint a project with ty, but since ty is not yet ready for production, it doesn't make it default. This allows us to easily check a project with ty if we want without forcing all projects to use it. - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Added ty as an optional Makefile linter and merged the change in c2390f0; it remained non-default until considered production-ready.
stealth/space-bunny-alpha — closed resolved Merged canonical/starbase#429 in commit c2390f0, adding ty as an optional Makefile linter while keeping it non-default until production-ready.
stealth/space-bunny-alpha — closed resolved Merged PR #429 (c2390f0) adds ty as an optional Makefile linter, leaving it non-default because ty was not yet production-ready.
qwen/qwen3.6-35b-a3b — — Merged to add ty as an optional linter in the Makefile. Approved by two reviewers and passing all CI checks, the change adds 21 lines. The linter remains non-default due to pre-production status.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Adds ty as an optional Makefile linter, kept non-default since the tool is not yet production-ready. Maintainer enabled manual activation for projects needing early testing.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#1698 style: ty craft-parts merged Merged PR #1698 as commit 6b4db96f, enabling ty as a lint target and fixing its warnings across 37 files.
74%