Skip to content

DOC: fix docstring validation errors for pandas.PeriodIndex #59638

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

Merged
merged 2 commits into from
Aug 28, 2024

Conversation

hlakams
Copy link
Contributor

@hlakams hlakams commented Aug 28, 2024

Addresses #59637.

Fixes:

pandas/ci/code_checks.sh

Lines 84 to 104 in 306385b

-i "pandas.PeriodIndex.day SA01" \
-i "pandas.PeriodIndex.day_of_week SA01" \
-i "pandas.PeriodIndex.day_of_year SA01" \
-i "pandas.PeriodIndex.dayofweek SA01" \
-i "pandas.PeriodIndex.dayofyear SA01" \
-i "pandas.PeriodIndex.days_in_month SA01" \
-i "pandas.PeriodIndex.daysinmonth SA01" \
-i "pandas.PeriodIndex.from_fields PR07,SA01" \
-i "pandas.PeriodIndex.from_ordinals SA01" \
-i "pandas.PeriodIndex.hour SA01" \
-i "pandas.PeriodIndex.is_leap_year SA01" \
-i "pandas.PeriodIndex.minute SA01" \
-i "pandas.PeriodIndex.month SA01" \
-i "pandas.PeriodIndex.quarter SA01" \
-i "pandas.PeriodIndex.qyear GL08" \
-i "pandas.PeriodIndex.second SA01" \
-i "pandas.PeriodIndex.to_timestamp RT03,SA01" \
-i "pandas.PeriodIndex.week SA01" \
-i "pandas.PeriodIndex.weekday SA01" \
-i "pandas.PeriodIndex.weekofyear SA01" \
-i "pandas.PeriodIndex.year SA01" \

@hlakams hlakams requested a review from mroeschke as a code owner August 28, 2024 06:51
…xisting series.PeriodIndex.qyear docstring validation fix
@mroeschke mroeschke added this to the 3.0 milestone Aug 28, 2024
@mroeschke mroeschke added the Docs label Aug 28, 2024
@mroeschke mroeschke merged commit 5285ff8 into pandas-dev:main Aug 28, 2024
51 checks passed
@mroeschke
Copy link
Member

Thanks @hlakams

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants