Skip to content

Commit 7453528

Browse files
Martozarclaude
andcommitted
fix: add pyarrow to test deps and restore direct imports
Add pyarrow>=16.1.0 to the test dependency group so arrow tests run in CI rather than being skipped. Restore direct pyarrow imports in the test file (importorskip workaround no longer needed). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent c746b39 commit 7453528

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

packages/gooddata-pandas/pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ test = [
6565
"python-dotenv~=1.0.0",
6666
"pyyaml",
6767
"tests_support",
68+
"pyarrow>=16.1.0",
6869
]
6970

7071
[tool.ty.analysis]

packages/gooddata-pandas/tests/dataframe/test_dataframe_for_exec_def_arrow.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,11 @@
77

88
import numpy
99
import pandas
10+
import pyarrow as pa
1011
import pytest
11-
12-
pa = pytest.importorskip("pyarrow", reason="pyarrow is required for Arrow tests")
13-
ipc = pytest.importorskip("pyarrow.ipc", reason="pyarrow is required for Arrow tests")
14-
15-
from gooddata_pandas._arrow_types import TypesMapper # noqa: E402
16-
from gooddata_pandas.arrow_convertor import compute_row_totals_indexes, convert_arrow_table_to_dataframe # noqa: E402
12+
from gooddata_pandas._arrow_types import TypesMapper
13+
from gooddata_pandas.arrow_convertor import compute_row_totals_indexes, convert_arrow_table_to_dataframe
14+
from pyarrow import ipc
1715

1816
_ARROW_FIXTURES = Path(__file__).parent / "fixtures" / "arrow"
1917

uv.lock

Lines changed: 2 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)