DOC: Fix quotes position in Series docstrings
State: closed

- [ ]xref #24059
- [ ] tests added / passed
- [ ] passes `git diff upstream/master -u -- "*.py" | flake8 --diff`
- [ ] whatsnew entry


Labels:
name: Docs
name: Code Style
description: Code style, linting, code_checks
name: Clean

Comments:
It's not done yet... for some, i did change the docstring, and i expect the output to change correspondingly, but result doesn't.. still need to find where the problem is... e.g. `pandas.series.dt.freq`, if you could help point out where i did wrong, I will be appreciated a lot because i was afraid if my direction was wrong or not.
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=h1) Report
> Merging [#24065](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/367efb280c69a55b676410bddc4497e744f90755?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `92.5%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/24065/graphs/tree.svg?width=650&token=eZ4WkYLtcO&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #24065   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files         161      161           
  Lines       51562    51562           
=======================================
  Hits        47599    47599           
  Misses       3963     3963
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.71% <92.5%> (ø)` | :arrow_up: |
| #single | `42.44% <48.75%> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/arrays/base.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYXJyYXlzL2Jhc2UucHk=) | `97.35% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/multi.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9tdWx0aS5weQ==) | `95.51% <ø> (ø)` | :arrow_up: |
| [pandas/core/dtypes/dtypes.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZHR5cGVzL2R0eXBlcy5weQ==) | `95.59% <ø> (ø)` | :arrow_up: |
| [pandas/plotting/\_core.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb3JlLnB5) | `83.58% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/period.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9wZXJpb2QucHk=) | `93.07% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/datetimelike.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9kYXRldGltZWxpa2UucHk=) | `97.41% <ø> (ø)` | :arrow_up: |
| [pandas/core/dtypes/base.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZHR5cGVzL2Jhc2UucHk=) | `100% <ø> (ø)` | :arrow_up: |
| [pandas/plotting/\_converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb252ZXJ0ZXIucHk=) | `63.67% <ø> (ø)` | :arrow_up: |
| [pandas/core/series.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvc2VyaWVzLnB5) | `93.68% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexing.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhpbmcucHk=) | `93.87% <ø> (ø)` | :arrow_up: |
| ... and [18 more](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=footer). Last update [367efb2...16d3462](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=h1) Report
> Merging [#24065](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/367efb280c69a55b676410bddc4497e744f90755?src=pr&el=desc) will **not change** coverage.
> The diff coverage is `92.5%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/24065/graphs/tree.svg?width=650&token=eZ4WkYLtcO&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=tree)

```diff
@@           Coverage Diff           @@
##           master   #24065   +/-   ##
=======================================
  Coverage   92.31%   92.31%           
=======================================
  Files         161      161           
  Lines       51562    51562           
=======================================
  Hits        47599    47599           
  Misses       3963     3963
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `90.71% <92.5%> (ø)` | :arrow_up: |
| #single | `42.44% <48.75%> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/core/arrays/base.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvYXJyYXlzL2Jhc2UucHk=) | `97.35% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/multi.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9tdWx0aS5weQ==) | `95.51% <ø> (ø)` | :arrow_up: |
| [pandas/core/dtypes/dtypes.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZHR5cGVzL2R0eXBlcy5weQ==) | `95.59% <ø> (ø)` | :arrow_up: |
| [pandas/plotting/\_core.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb3JlLnB5) | `83.58% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/period.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9wZXJpb2QucHk=) | `93.07% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexes/datetimelike.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhlcy9kYXRldGltZWxpa2UucHk=) | `97.41% <ø> (ø)` | :arrow_up: |
| [pandas/core/dtypes/base.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZHR5cGVzL2Jhc2UucHk=) | `100% <ø> (ø)` | :arrow_up: |
| [pandas/plotting/\_converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb252ZXJ0ZXIucHk=) | `63.67% <ø> (ø)` | :arrow_up: |
| [pandas/core/series.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvc2VyaWVzLnB5) | `93.68% <ø> (ø)` | :arrow_up: |
| [pandas/core/indexing.py](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvaW5kZXhpbmcucHk=) | `93.87% <ø> (ø)` | :arrow_up: |
| ... and [18 more](https://codecov.io/gh/pandas-dev/pandas/pull/24065/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=footer). Last update [367efb2...16d3462](https://codecov.io/gh/pandas-dev/pandas/pull/24065?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

@datapythonista is there a lint rule to enforce this?
yes, we've got `GL01` and `GL02` in the validation script for that. But we'll be able to have in the CI after a follow up issue of this one, as the issue was to fix around 50% of the cases. This will be the case with several of the issues, and my plan is to fix all them to be able to validate the errors in the CI. So, you can leave it with me, I'll ask to add the validation in the issues where this is possible.
@datapythonista yeah... indeed, i was thinking to add line breaks for docstring `f.__doc__ = docstring` directly in `_field_accessor` function, but looks it couldn't solve all errors... since i found out some of error are due to lack of docstring in some methods...
We'll focus on the special cases once we have the rest fixed. There were around 250 wrong cases of the error you've been fixing. I'll open an issue for the remaining once this is merged.
thanks @charlesdong1991 
@datapythonista ok, i could push the new change for the `_field_accessor` to solve the issue you mentioned above if you want. i had the change in the local, i didn't add it since i was not sure if it's what you want...
sure, can you open another PR for it? And if you can check the the calls to `_field_accessor` and make sure all the line breaks (and spaces at the beginning or the end) are removed, that would be great. The issue is #24068