remove pyarrow dependency#63
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes the pyarrow dependency from the project's core dependencies, simplifying the package requirements while maintaining the draken dependency.
- Removed
pyarrowfrom the required dependencies list
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| dependencies = [ | ||
| "draken", | ||
| "pyarrow", | ||
| "draken" |
There was a problem hiding this comment.
Removing pyarrow may break existing functionality that depends on it. Ensure that either: (1) pyarrow is no longer used anywhere in the codebase, (2) it's been moved to optional-dependencies if still needed for certain features, or (3) the draken package now includes pyarrow as its own dependency. The keywords list still includes 'parquet' which typically requires pyarrow, suggesting this dependency may still be needed.
| "draken" | |
| "draken", | |
| "pyarrow" |
No description provided.