-
-
Notifications
You must be signed in to change notification settings - Fork 533
Fix ^C during venv creation #1258
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 ^C during venv creation #1258
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok but pytest fails on Windows 2.7 with a not that helpful error 🤔
It passed before, I think this is a flake / issue with pytest-xdist / pytest -- can you rerun it? |
Triggered a rerun 👍 |
What's the flake issue with pytest-xdist? Still fails 😢 |
It has to do with printing non-ascii characters in error messages during pytest failures and multiprocessing in python2.x specifically on windows -- I've had a rough time trying to reproduce it myself but here's the stacktrace (I've seen this before):
|
looks like it didn't trigger, I'll just trivial-amend |
It's still failing on master, for some reason now reliably. Maybe a side-effect of the Azure image upgrade. Either way, we need to fix now to merge further PRs. 😢 |
Resolves #1257