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

Install and run pytype on Windows and Python 3.12 #12547

Closed
wants to merge 18 commits into from

Conversation

Avasam
Copy link
Collaborator

@Avasam Avasam commented Aug 18, 2024

I'm expecting to have issues, but since it's been updated for Python 3.12 and Windows in the last week I figured may as well try it out.

Edit: Actually looks pretty good! This could work as-is.

@Avasam Avasam marked this pull request as draft August 18, 2024 06:50
@Avasam Avasam changed the title Install and run unreleased pytype on windows and Python 3.12 Install and run unreleased pytype on Windows and Python 3.12 Aug 18, 2024
@Avasam
Copy link
Collaborator Author

Avasam commented Aug 18, 2024

Comment on lines 169 to 173
assert len(spec) == 1, f"type checker/linter '{package}' has complex specifier in {REQS_FILE}"
if package != "pytype": # Temporarily allow complex specification for pytype
assert len(spec) == 1, f"type checker/linter '{package}' has complex specifier in {REQS_FILE}"
url = requirements[package].url
msg = f"type checker/linter '{package}' is not pinned to an exact version in {REQS_FILE}"
assert str(spec).startswith("=="), msg
assert str(spec).startswith("==") or "@" in str(url), msg
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This adds support to target a specific git commit for install. But won't be necessary once an actual pytype version releases.

tests/pytype_exclude_list.txt Outdated Show resolved Hide resolved
@frigus02
Copy link
Contributor

We released pytype 2024.09.13 today, which includes basic Python 3.12 support.

Windows also likely works but we can't enable our tests for it, yet, because they run too slowly. We'll have to do some more work there before we can officially declare that pytype supports Windows. Not sure if you want to run tests on Windows before that.

I opened #12653, which only bumps the pytype version and leaves the remaining caveats unchanged. I don't mind which one gets merged, though.

@Avasam Avasam changed the title Install and run unreleased pytype on Windows and Python 3.12 Install and run pytype on Windows and Python 3.12 Sep 16, 2024
@Avasam Avasam deleted the pytype-on-windows-and-Python-3.12 branch September 17, 2024 12:12
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.

3 participants