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.

Sign up for GitHub

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

Jump to bottom

ci(typing): Add pyarrow-stubs to dev dependencies #3679

Merged
merged 1 commit into from
Nov 12, 2024
Merged

ci(typing): Add pyarrow-stubs to dev dependencies #3679

merged 1 commit into from
Nov 12, 2024

Conversation

dangotbanned
Copy link
Member

@dangotbanned dangotbanned commented Nov 12, 2024 •
edited
Loading

Related 0bb4210

https://github.com/zen-xu/pyarrow-stubs

In #3631 this stub package has been a huge help in understanding pyarrow.

Wanted to split out this dev dependency change for visibility.

Also, the 1 minor change to satisfy mypy on a test:

tests\utils\test_utils.py:143: error: Argument 1 to "schema" has incompatible type "list[object]"; expected "Iterable[Field[Any]] | Iterable[tuple[str, DataType]] | Mapping[str, DataType]"  [arg-type]

@dangotbanned dangotbanned added maintenance dependencies Pull requests that update a dependency file labels Nov 12, 2024
@dangotbanned dangotbanned enabled auto-merge (squash) November 12, 2024 19:49
@dangotbanned dangotbanned merged commit b4816e9 into main Nov 12, 2024
25 checks passed
@dangotbanned dangotbanned deleted the dep-pyarrow-stubs branch November 12, 2024 20:21
@dangotbanned dangotbanned mentioned this pull request Nov 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file maintenance
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dangotbanned