-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Added new functions to unregister readers, writers, and identifiers #6217
Conversation
CHANGES.rst
Outdated
@@ -56,6 +56,8 @@ New Features | |||
|
|||
- ``astropy.io.registry`` | |||
|
|||
- New functions to unregister readers, writers, and identifiers. [#6216] |
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.
Off by one!
Do we need What's New for this?
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.
Oops: astropy/astropy-tools@127fffc 😊
Don't think we need a what's new, I don't think this is really exciting enough to qualify.
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.
LGTM except for one minor thing. I'll fix that online and then merge.
astropy/io/registry.py
Outdated
_identifiers.pop((data_format, data_class)) | ||
else: | ||
raise IORegistryError("No identifier defined for format '{0}' and class '{1}'" | ||
''.format(data_format, data_class.__name__)) |
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.
(nitpick) >80 chars
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'll just fix this directly and merge.
[ci skip]
@taldcroft @MSeifert04 - I think you contributed the most edits to this file, so tagging you so you can review this. I'd like to get this in 2.0 if it's uncontroversial.