Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Added Warning and Exception annotations #57

Merged
merged 3 commits into from
Apr 18, 2020
Merged

Added Warning and Exception annotations #57

merged 3 commits into from
Apr 18, 2020

Conversation

BvB93
Copy link
Member

@BvB93 BvB93 commented Apr 18, 2020

See #54.

Added annotations and tests for:

Bas van Beek added 2 commits April 18, 2020 16:13
See https://github.com/numpy/numpy-stubs/issues/54.

Added annotations and tests for:
* np.warnings
* np.ModuleDeprecationWarning
* np.VisibleDeprecationWarning
* np.ComplexWarning
* np.RankWarning
* np.TooHardError
* np.AxisError
Copy link
Member

@person142 person142 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I had a few comments (and black is unhappy), but it looks good!

numpy-stubs/__init__.pyi Outdated Show resolved Hide resolved
numpy-stubs/__init__.pyi Outdated Show resolved Hide resolved
tests/pass/warnings_and_errors.py Show resolved Hide resolved
tests/pass/warnings_and_errors.py Outdated Show resolved Hide resolved
* Removed ``warnings`` (#57 (comment))
* Deleted a now redundant comment (#57 (comment))
* Created and/or moved a number of tests to fail/ and reveal/ (#57 (comment) and #57 (comment))
* Formatted the main __init__.pyi file with black (#57 (review))
@BvB93
Copy link
Member Author

BvB93 commented Apr 18, 2020

Thanks for feedback! I've addressed your points in the latest push.

@person142 person142 merged commit 559ff04 into numpy:master Apr 18, 2020
@person142
Copy link
Member

Great, thanks @BvB93.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants