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
Restore numba.types
as public API
#5676
Conversation
"""Testing moved modules in Q1 2020 but were decided to kept as public API | ||
""" | ||
def tests_numba_types(self): | ||
import numba.types |
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.
Should e.g. from numba.types import containers
be explicitly tested too?
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 can add a test. In general, I am only testing a few things because I am already checking for object identity. I don't think that's any reason for it to not work.
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.
test added in 2998661
That error on macos is odd. |
This is unexpected: In [1]: import numba.types.misc
In [2]: numba.types.misc.StringLiteral
Out[2]: numba.types.misc.StringLiteral
In [3]: from numba.core import types
In [4]: types.StringLiteral
Out[4]: numba.core.types.misc.StringLiteral |
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 for the patch. I tried this out manually, couldn't break it!
and remove outdated test test_numba_types
as titled.