-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: docstring cleanup #40713
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: docstring cleanup #40713
Conversation
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.
minor comment for followup
@@ -2412,7 +2412,7 @@ def isna(self) -> np.ndarray: | |||
|
|||
Returns | |||
------- | |||
numpy.ndarray | |||
numpy.ndarray[bool] |
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.
completely minor but i think inconsistent as use of numpy.ndarray
vs np.darray
(see this much more) vs ndarray
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 you're right, decided against trying to make these uniform in this PR
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.
green
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.
yep next one (and prob should have a rule for this). note that I am saying it out loud but not knowing how hard that is :->
No description provided.