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

Add option to keep tmp dir to stubtest script #12151

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

srittau
Copy link
Collaborator

@srittau srittau commented Jun 17, 2024

Also add prefix to tmpdir name

@srittau
Copy link
Collaborator Author

srittau commented Jun 17, 2024

I'm very confused about the mypy error on win32. Can anyone shed a light on what I did wrong?

https://github.com/python/typeshed/actions/runs/9547845403/job/26313973228?pr=12151

@Avasam
Copy link
Sponsor Collaborator

Avasam commented Jun 26, 2024

I'm very confused about the mypy error on win32. Can anyone shed a light on what I did wrong?

python/typeshed/actions/runs/9547845403/job/26313973228?pr=12151

idk, the error doesn't even look related to your changes, and the pre-existing check looks correct. Maybe you found a new false-positive by wrapping in an additional try-except block?

@srittau
Copy link
Collaborator Author

srittau commented Jul 12, 2024

This is python/mypy#14309. I will disable "possibly-undefined" for now.

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.

None yet

2 participants