-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLEAN dubious docstring #50344
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
Comments
Added in #48011 |
thanks @asishm
Do you want to submit a PR to fix it up? |
The initial docstring also won't show up - it's already present after a code-block pandas/pandas/_libs/tslibs/offsets.pyx Lines 369 to 375 in 22491dc
n, normalize are not properties (so there can't be any docstrings) - they're plain attributes (unless i'm reading it wrong). They are also documented here - pandas/pandas/_libs/tslibs/offsets.pyx Line 1314 in 22491dc
|
yeah what I mean is to move the docstring to be directly under |
take |
This looks odd - the docstring is broken in two, and there's an assignment in the middle?
pandas/pandas/_libs/tslibs/offsets.pyx
Lines 369 to 389 in 22491dc
It looks like something might not have gone quite right during a merge conflict, and was kept in?
If anyone felt like having a look at
git blame
/git log
to figure this out, that'd be helpfulNo need to comment "can I work on this?" - if you know your way around
git
and figure this out, then go ahead and work on it - thanks 🙏The text was updated successfully, but these errors were encountered: