-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: removed pandas.sandbox #13670
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
CLN: removed pandas.sandbox #13670
Conversation
Current coverage is 84.51%@@ master #13670 diff @@
==========================================
Files 142 141 -1
Lines 51223 51142 -81
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 43224 43224
+ Misses 7999 7918 -81
Partials 0 0
|
@@ -110,78 +110,6 @@ details. | |||
Visualizing Data in Qt applications |
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.
How about removing whole section, and moves to ecosystem?
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.
So remove the entire thing about Qt applications? I don't fully understand the second part of your question.
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.
Yes. The second part means ecosystem.rst which lists some related projects.
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.
Got it. Done.
@sinhrks : made the requested doc comments (no need to run Travis tests again hence the |
[ci skip]
thank you sir! |
Deprecated back in
0.16.0
here, and the module seems to be rarely, if at all, used according to @jorisvandenbossche .