Skip to content

DOC: fix docstring of timedaltas.ceil #60047

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 30, 2024
Merged

DOC: fix docstring of timedaltas.ceil #60047

merged 2 commits into from
Oct 30, 2024

Conversation

eightyseven
Copy link
Contributor

@eightyseven eightyseven commented Oct 15, 2024 •
edited
Loading

@eightyseven
Copy link
Contributor Author

eightyseven commented Oct 15, 2024 •
edited
Loading

I check the function timedelta.ceil() and find that only D,H,min,s,ms,us,ns are valid inputs but maybe more like 'm' will be added later?so I didn't list all the possilble value in the docstring

mroeschke
mroeschke reviewed Oct 29, 2024
View reviewed changes
@mroeschke mroeschke added the Docs label Oct 29, 2024
@mroeschke mroeschke mentioned this pull request Oct 29, 2024
Closed
2 tasks
@mroeschke mroeschke added this to the 3.0 milestone Oct 30, 2024
@mroeschke mroeschke merged commit 71eb457 into pandas-dev:main Oct 30, 2024
49 of 51 checks passed
@mroeschke
Copy link
Member

Thanks @eightyseven

@eightyseven eightyseven deleted the doc-change-3 branch November 7, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mroeschke mroeschke mroeschke approved these changes

@MarcoGorelli MarcoGorelli Awaiting requested review from MarcoGorelli

Assignees

No one assigned

Labels

Docs

Projects

None yet

Milestone

3.0

Development

Successfully merging this pull request may close these issues.

docs: update freq description for pd.Timedelta.ceil to match DatetimeIndex.ceil's freq description

2 participants

@eightyseven @mroeschke