Skip to content
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

DOC: Remove redundant table in Series docs #37143

Merged
merged 1 commit into from
Oct 16, 2020

Conversation

erictleung
Copy link
Contributor

The attribute Series.index was separated in another table, which makes
the rendering of the documentation slightly inconsistent. This commit
consolidates the tables together to make the documentation consistent
and aligned.

Live site (https://pandas.pydata.org/docs/reference/series.html):

image

Locally rendered change:

image

I'm not sure why the color changed. My guess is because I just rendered that page with python make.py --single reference/series.rst, so the links to those pages are nonexistent.

If this separate table was intentional, feel free to close this PR. Thanks for considering this!

  • closes #xxxx
  • tests added / passed
  • passes black pandas
  • passes git diff upstream/master -u -- "*.py" | flake8 --diff
  • whatsnew entry

The attribute `Series.index` was separated in another table, which makes
the rendering of the documentation slightly inconsistent. This commit
consolidates the tables together to make the documentation consistent
and aligned.
@jreback jreback added the Docs label Oct 16, 2020
@jreback jreback added this to the 1.2 milestone Oct 16, 2020
@jreback jreback merged commit e7ee7cb into pandas-dev:master Oct 16, 2020
@jreback
Copy link
Contributor

jreback commented Oct 16, 2020

good catch thanks @erictleung

JulianWgs pushed a commit to JulianWgs/pandas that referenced this pull request Oct 26, 2020
kesmit13 pushed a commit to kesmit13/pandas that referenced this pull request Nov 2, 2020
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.

None yet

2 participants