Skip to content

Conversation

@nanxstats
Copy link
Collaborator

This PR removes pandas and pyarrow from the development dependencies list in pyproject.toml and runs uv sync.

Looks like they can now be safely removed without stopping any content in the code/tests/docs from running. If so, it would make sense to not keep them there - we should be a pure polars shop for the backend and a narwhals shop for the frontend.

Hopefully I'm not missing or breaking anything important.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes pandas and pyarrow from the development dependencies, aligning with the project's goal to be a "pure polars shop for the backend and a narwhals shop for the frontend." The removal is safe as these libraries are no longer imported or used anywhere in the codebase.

Changes:

  • Removed pandas and pyarrow from the development dependencies in pyproject.toml
  • Updated uv.lock to reflect the dependency changes
  • Added changelog entries documenting the dependency removal

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

File Description
pyproject.toml Removed pandas (>=2.2.3) and pyarrow (>=21.0.0/>=22.0.0) from the dev dependency group
uv.lock Removed pandas 2.3.3 and pyarrow 22.0.0 package definitions and their transitive dependencies (numpy, python-dateutil, pytz, tzdata)
docs/changelog.md Added entry documenting the removal of pandas and pyarrow from development dependencies
CHANGELOG.md Added entry documenting the removal of pandas and pyarrow from development dependencies

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

@elong0527 elong0527 merged commit 59b703c into main Jan 12, 2026
15 checks passed
@nanxstats nanxstats deleted the pandas branch January 12, 2026 22:31
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.

2 participants