-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
STYLE: inconsistent namespace (io, reshape) #40069
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
STYLE: inconsistent namespace (io, reshape) #40069
Conversation
AA-Turner
commented
Feb 25, 2021
•
edited
Loading
edited
- xref STYLE Increase coverage of inconsistent-namespace-usage #39992
- Ensure all linting tests pass, see here for how to run them
@MarcoGorelli this is Thanks! |
can you merge master |
850c045
to
8ea4fc2
Compare
Rebased & pushed |
Hi @AA-Turner - |
Thanks Marco, mutst've missed that when I was looking for what had been done already. Will do as you ask |
8ea4fc2
to
3048a86
Compare
Done |
Thanks @AA-Turner ! |
Looks like this PR had some failure - seeing them on master as well now. Are these related? |
Hi Richard - will look into this. Edit: found the cause, see below Given a lot of tests/CI checks failed - is there a known subset that are ingored? I'll look for |
Found three failures I think are related - all in PR to fix is at #40094 Adam |