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
use importlib.machinery when available #4928
Conversation
import inspect | ||
import os | ||
import re | ||
import sys | ||
|
||
try: |
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.
Do we want some comments here for which versions of Python take which paths?
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.
Yup, noted.
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.
This comment should mention Python >=3.3, not just 3, since e.g. 3.2 does not have importlib.machinery.all_suffixes
.
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, fixed in 40495fa.
avoids deprecation warning on startup, and allows tests to pass with PYTHONWARNINGS=d closes ipython#4051 candidate for backport to 1.2
use importlib.machinery when available
Backported |
You forgot to change |
Oops, thanks @Arfrever. Should be fixed now. |
use importlib.machinery when available
avoids deprecation warning on startup, and allows tests to pass with
PYTHONWARNINGS=d
closes #4051
candidate for backport to 1.2