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: ensure is_windows, detect_windows_subsystem, uname work under MSYS/Cygwin #4313
Conversation
b78e398
to
8db4362
Compare
…Cygwin shells Signed-off-by: SSE4 <tomskside@gmail.com>
8db4362
to
6916dac
Compare
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.
In love with unittests! 😍 Thanks!
Signed-off-by: SSE4 <tomskside@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
Signed-off-by: SSE4 <tomskside@gmail.com>
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.
👍 Thanks!
it failed with weird error |
@@ -0,0 +1,3 @@ | |||
#!/usr/bin/env python | |||
# -*- coding: utf-8 -*- | |||
# vim: tabstop=8 expandtab shiftwidth=4 softtabstop=4 |
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.
Hi @SSE4
Please try to avoid adding this vim
-only line to all the files.
closes #4312
Changelog: Fix: ensure is_windows, detect_windows_subsystem, uname work under MSYS/Cygwin
Docs: omit
develop
branch, documenting this one.Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.