Skip to content

ty as yet another type checker - #5058

Open
happz wants to merge 1 commit into
mainfrom
pre-commit-ty
Open

ty as yet another type checker#5058
happz wants to merge 1 commit into
mainfrom
pre-commit-ty

Conversation

@happz

@happz happz commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Still heavily reduced ruleset, but over time, things should improve.

Pull Request Checklist

  • implement the feature

Still heavily reduced ruleset, but over time, things should improve.
@happz happz added the code | type annotations Related to type annotations and type cleanup label Jul 14, 2026
@happz happz added this to planning Jul 14, 2026
@happz happz added the test coverage Improvements or additions to test coverage of tmt itself label Jul 14, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Jul 14, 2026
@happz happz added the ci | full test Pull request is ready for the full test execution label Jul 14, 2026
@happz
happz marked this pull request as ready for review July 14, 2026 06:35

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request integrates the ty pre-commit hook, configures its rules in pyproject.toml, and updates the return type of execute in tmt/guest/__init__.py to be optional. Correct the typo call-top-callable to call-non-callable in the pyproject.toml configuration.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread pyproject.toml
invalid-yield = "ignore"
possibly-missing-submodule = "ignore"
invalid-generic-class = "ignore"
call-top-callable = "ignore"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Correct the typo call-top-callable to call-non-callable.

Suggested change
call-top-callable = "ignore"
call-non-callable = "ignore"

@happz happz moved this from backlog to review in planning Jul 14, 2026

@LecrisUT LecrisUT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should start type-checking the tests also at some point

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution code | type annotations Related to type annotations and type cleanup test coverage Improvements or additions to test coverage of tmt itself

Projects

Status: review

Development

Successfully merging this pull request may close these issues.

2 participants