-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
Skip doctest on 32bits arch #486
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.
I think this needs a black
run to fix formatting as well.
sparse/tests/conftest.py
Outdated
|
||
def pytest_cmdline_preparse(args): | ||
if platform.system() != "Windows": | ||
args.append("--doctest-modules") | ||
if sysinfo.platform_bits != 32: |
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.
I think this should be in the first if with an or.
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.
Done!
If you use |
I tried |
You're right. My mistake. |
@hameerabbasi Thanks for the merge! :-) Does it make sense to enable CI for a 32bits arch as well? |
Disable doctest on 32bits architectures because they generate innocent errors.
See #428 (comment) and https://bugs.debian.org/988424
Now, during a build on a 32bits system, we have other errors: