-
Notifications
You must be signed in to change notification settings - Fork 85
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
Deprecate SingletonHasTraits & related classes #887
Conversation
Deprecated Classes | ||
------------------ | ||
|
||
The following :class:`~.HasTraits` classes and instances are deprecated, |
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 think we could replace "classes and instances" with just "subclasses" here.
This LGTM as far as it goes. Do you think we can find a way for these classes to issue a |
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.
Can we look into ways to have these issue a DeprecationWarning
on use?
dc5d07f
to
b0c1b98
Compare
Made the doc changes. Had to force push after messing up the history. |
Codecov Report
@@ Coverage Diff @@
## master #887 +/- ##
=========================================
+ Coverage 72.58% 72.89% +0.3%
=========================================
Files 51 51
Lines 6471 6474 +3
Branches 1302 1302
=========================================
+ Hits 4697 4719 +22
+ Misses 1378 1355 -23
- Partials 396 400 +4
Continue to review full report at Codecov.
|
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. The deprecation warning gets issued at instantiation time rather than subclassing time, which is fine. (We probably could find a way to issue it at subclassing time, but then we'd get DeprecationWarnings being issued in imports, which is always awkward to deal with.)
Fix #881
PR updates creates a Deprecated Classes section in the has_traits docs.