DOC: Fix PEP-8 issues in timedeltas.rst

[FHaase]

• Replace from pandas.tseries.offsets import * with pd.offsets.<ClassName> for better readability.

[codecov]

Codecov Report

Merging #23905 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #23905   +/-   ##
=======================================
  Coverage   92.29%   92.29%           
=======================================
  Files         161      161           
  Lines       51498    51498           
=======================================
  Hits        47530    47530           
  Misses       3968     3968

Flag	Coverage Δ
#multiple	90.69% <ø> (ø)	⬆️
#single	42.43% <ø> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7b187a...6bc4ce5. Read the comment docs.

[datapythonista]

lgtm, thanks @FHaase

@jbrockmendel if you want to take a look here too

[jbrockmendel]

This looks fine to me. Only question is if the importing of Second/Minute etc is intentional.

[datapythonista]

Not sure what you mean. Replacing the from pandas.tseries.offsets import * and using the full path version of Second... is intentional, as importing * is discouraged in Python, and having the full path makes things clearer IMO, as the user know where Second and the rest are coming from. Is this what you mean?

[jbrockmendel]

I would have a strong preference against * imports in actual python code, but the .rst files I'm not so familiar with, don't know if separate conventions apply there.

[FHaase]

Well if someone reads the documentation finding something like Second(...) it's not clear what that class is. Is it a pandas class, where in the pandas project can it be found. Is it defined somewhere in the previous *.rst file.

By writing pd.offsets.Second(...) it's basically clear on first glance.

[jreback]

thanks @FHaase

yeah more explicit the better.