Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: improve type resolutions #1016

Merged
merged 5 commits into from
Aug 3, 2024

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented Jul 31, 2024

Running pyright yields many typing violations - some indicating actual issues and some due to lack of hints.

This does a minor refactor top remove a lot of the false-positive errors.

No user-facing changes in this PR. Actual bugfixes will be submitted as follow-ups after merge of this PR.

@legobeat legobeat force-pushed the refactor-resolvable-type-checks branch from 96a3306 to 7007946 Compare July 31, 2024 07:11
@legobeat legobeat force-pushed the refactor-resolvable-type-checks branch from 9904196 to 4615d20 Compare July 31, 2024 07:22
podman_compose.py Outdated Show resolved Hide resolved
@legobeat legobeat force-pushed the refactor-resolvable-type-checks branch from 4615d20 to c900d6d Compare August 1, 2024 23:10
@legobeat legobeat requested a review from p12tic August 1, 2024 23:49
@legobeat legobeat force-pushed the refactor-resolvable-type-checks branch 2 times, most recently from 77d9b42 to 0435a57 Compare August 2, 2024 23:52
When support for python3.8 and python3.9 has been dropped, it will be possible to eat the cak and
have it due to PEP-647.

Signed-off-by: legobt <[email protected]>
@legobeat legobeat force-pushed the refactor-resolvable-type-checks branch from 0435a57 to ea239c4 Compare August 3, 2024 01:01
@p12tic p12tic merged commit eeefd37 into containers:main Aug 3, 2024
8 checks passed
@legobeat legobeat deleted the refactor-resolvable-type-checks branch August 3, 2024 14:58
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