Skip to content

Fixing all mypy errors#116

Merged
pkienzle merged 19 commits intofix-104-type-hintingfrom
fixing-all-mypy-errors
Feb 18, 2026
Merged

Fixing all mypy errors#116
pkienzle merged 19 commits intofix-104-type-hintingfrom
fixing-all-mypy-errors

Conversation

@bmaranville
Copy link
Collaborator

@bmaranville bmaranville commented Feb 17, 2026

This is still in draft form... I get 33 remaining errors when running mypy.

Warning: base set fix-104-type-hinting for review. Either merge with fix-104-type-hinting, or replace with master after fix-104-type-hinting is merged.

@pkienzle pkienzle changed the base branch from master to fix-104-type-hinting February 18, 2026 01:57
@pkienzle
Copy link
Collaborator

Strange that it is not finding pytest-mypy. I see it in the dev dependencies. Maybe the wheel builder isn't using dev?

@pkienzle pkienzle marked this pull request as ready for review February 18, 2026 02:24
@pkienzle pkienzle marked this pull request as draft February 18, 2026 02:25
@pkienzle
Copy link
Collaborator

pkienzle commented Feb 18, 2026

I fixed a mypy error and now it passes cleanly on my machine. I don't know why CI isn't triggered in the PR.


Update: CI was only running when base was master, so it stopped when I set base to fix-104-type-hinting

@pkienzle pkienzle marked this pull request as ready for review February 18, 2026 05:32
@pkienzle pkienzle merged commit b63757d into fix-104-type-hinting Feb 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants