Skip to content

DOC: fixing GL08 errors for pandas.DatetimeIndex.as_unit and pandas.DatetimeIndex.freq #57562

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 10 commits into from
Feb 26, 2024

Conversation

jordan-d-murphy
Copy link
Contributor

@jordan-d-murphy jordan-d-murphy commented Feb 22, 2024

All GL08 Errors resolved in the following cases:

  1. scripts/validate_docstrings.py --format=actions --errors=GL08 pandas.DatetimeIndex.as_unit
  2. scripts/validate_docstrings.py --format=actions --errors=GL08 pandas.DatetimeIndex.freq

Note: 6232a2e commit message was a typo, this commit is for GL08 errors

@mroeschke mroeschke added the Docs label Feb 22, 2024
@jordan-d-murphy
Copy link
Contributor Author

pre-commit.ci autofix

Copy link
Contributor Author

@jordan-d-murphy jordan-d-murphy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made a few updates to get the CI checks passing.

@jordan-d-murphy
Copy link
Contributor Author

pre-commit.ci autofix

@mroeschke mroeschke added this to the 3.0 milestone Feb 26, 2024
@mroeschke mroeschke merged commit 1f860f3 into pandas-dev:main Feb 26, 2024
@mroeschke
Copy link
Member

Thanks @jordan-d-murphy

@jordan-d-murphy jordan-d-murphy deleted the issue#57443_1 branch February 26, 2024 18:24
pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
…atetimeIndex.freq (pandas-dev#57562)

* fixing PR01 errors for pandas.DatetimeIndex.as_unit and pandas.DatetimeIndex.freq

* clean up pandas.DatetimeIndex.freq docstring validation errors

* update link to use :ref: format

* improving docstring for as_unit

* add missing period

* add missing period

* Specify ValueError

Co-authored-by: Jonas Bergner <[email protected]>

* cleaning up

---------

Co-authored-by: Jonas Bergner <[email protected]>
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.

3 participants