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.

Sign up for GitHub

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

Jump to bottom

Enable and fix Python warnings #1058

Closed
giampaolo opened this issue May 10, 2017 · 1 comment
Closed

Enable and fix Python warnings #1058

giampaolo opened this issue May 10, 2017 · 1 comment
Labels
enhancement tests

Comments

@giampaolo
Copy link
Owner

Test suite should be run with PYTHONWARNINGS=all or -Wa cmdline option in order to reveal warnings. Makefile should be modified in accordance and all the warnings should be fixed.
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: have Makefile use PYTHONWARNINGS=all by default for (almost) a…
d63a1e3 
…ll commands
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058 fix linux tests warnings
a26da98
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: try not to use imp module
3fdd505
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: get rid of imp module completely
9e5c3b0
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: ignore unicode warnings
4823c3a
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: ignore stderr from procsmem.py
a250dbc
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: fix resource warning from Popen
f248307
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
#1058: get rid of contextlib.nested (deprecated)
65ea7a8
giampaolo added a commit that referenced this issue May 10, 2017
@giampaolo
1058 enable fix warnings (#1059)
58aef46 
* #1058: have Makefile use PYTHONWARNINGS=all by default for (almost) all commands

* #1058 fix linux tests warnings

* #1058: try not to use imp module

* #1058: get rid of imp module completely

* #1058: ignore unicode warnings

* #1058: ignore stderr from procsmem.py

* #1058: fix resource warning from Popen

* #1058: get rid of contextlib.nested (deprecated)
@giampaolo giampaolo added the bug label May 10, 2017
@giampaolo
Copy link
Owner Author

PR: #1059

giampaolo added a commit that referenced this issue May 12, 2017
@giampaolo
#1058: enable warnings on make.bat, appveyor and travis
b258d82
@giampaolo giampaolo added tests and removed bug labels Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@giampaolo