Skip to content

DOC: fix copy-paste bug in PyArrow storage example in text.rst - #66572

Merged
jbrockmendel merged 1 commit into
pandas-dev:mainfrom
rishitoshsingh:doc-fix-text-storage-value
Aug 1, 2026
Merged

DOC: fix copy-paste bug in PyArrow storage example in text.rst#66572
jbrockmendel merged 1 commit into
pandas-dev:mainfrom
rishitoshsingh:doc-fix-text-storage-value

Conversation

@rishitoshsingh

@rishitoshsingh rishitoshsingh commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

The "PyArrow storage with pd.NA values" example in text.rst used
storage="python"; identical to the preceding "Python storage" example
and contradicting the section heading/text, which claims a PyArrow-backed
array. Fixed to storage="pyarrow".
No issue filed; this is a trivial one-word typo fix, which the
contributing guide exempts from requiring an associated issue.

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.
  • I have reviewed and followed all the contribution guidelines
  • If I used AI to develop this pull request, I prompted it to follow AGENTS.md.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Fixes a copy/paste error in the user guide’s “PyArrow storage with pd.NA values” example so it actually constructs a PyArrow-backed StringDtype (matching the section heading and narrative).

Changes:

  • Update the example pd.StringDtype(...) to use storage="pyarrow" instead of storage="python" in doc/source/user_guide/text.rst.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jbrockmendel
jbrockmendel merged commit 2bc326d into pandas-dev:main Aug 1, 2026
9 checks passed
@jbrockmendel

Copy link
Copy Markdown
Member

thanks @rishitoshsingh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants