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 InferType and improve type-checker tests #3382

Merged
merged 9 commits into from Jun 25, 2022
Merged

Fix InferType and improve type-checker tests #3382

merged 9 commits into from Jun 25, 2022

Conversation

rsokl
Copy link
Contributor

@rsokl rsokl commented Jun 25, 2022

Closes #3379

cb05529 and db5ed42 should make the test suite fail. (our pyright tests would fail for db5ed42 if we were using the pyright 1.1.250 or above)

@rsokl rsokl added tests/build/CI about testing or deployment *of* Hypothesis legibility make errors helpful and Hypothesis grokable labels Jun 25, 2022
@rsokl rsokl requested a review from Zac-HD June 25, 2022 16:06
Copy link
Member

@Zac-HD Zac-HD left a comment

Choose a reason for hiding this comment

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

Looks good - can you please also update the pinned Python version in requirements/tools.txt to confirm this fixes the problem?

@@ -16,6 +16,8 @@
from hypothesistooling.projects.hypothesispython import PYTHON_SRC
from hypothesistooling.scripts import pip_tool, tool_path

PYTHON_VERSIONS = [f"3.{v}" for v in range(7, 11)]
Copy link
Member

Choose a reason for hiding this comment

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

I think I'd prefer to write these out as literals, unless we can pull them out of the autoupdated CI config? Just thinking about how they'll stay up to date. I think we can also test against 3.11?

@rsokl
Copy link
Contributor Author

rsokl commented Jun 25, 2022

pinned Python version

You mean pinned pyright version, right?

@Zac-HD Zac-HD merged commit f00ef0d into master Jun 25, 2022
@Zac-HD Zac-HD deleted the fix-InferType branch June 25, 2022 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
legibility make errors helpful and Hypothesis grokable tests/build/CI about testing or deployment *of* Hypothesis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Signature of given is partially unknown for Python < 3.10
2 participants