Skip to content

Commit d3c5740

Browse files
committed
Remove PYARROW_TEST_ANNOTATIONS flag
1 parent dcd6d8c commit d3c5740

File tree

3 files changed

+11
-20
lines changed

3 files changed

+11
-20
lines changed

.github/workflows/python.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -241,8 +241,6 @@ jobs:
241241
run: ci/scripts/python_test.sh $(pwd) $(pwd)/build
242242
- name: Test annotations
243243
shell: bash
244-
env:
245-
PYARROW_TEST_ANNOTATIONS: "ON"
246244
run: ci/scripts/python_test_type_annotations.sh $(pwd)/python
247245

248246
windows:

ci/scripts/python_test_type_annotations.sh

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,21 +20,18 @@
2020
set -ex
2121
pyarrow_dir=${1}
2222

23-
if [ "${PYARROW_TEST_ANNOTATIONS}" == "ON" ]; then
24-
# Install library stubs
25-
pip install pandas-stubs scipy-stubs sphinx types-cffi types-psutil types-requests types-python-dateutil
23+
# Install library stubs
24+
pip install pandas-stubs scipy-stubs sphinx types-cffi types-psutil types-requests types-python-dateutil
2625

27-
# Install type checkers
28-
pip install mypy pyright ty
26+
# Install type checkers
27+
pip install mypy pyright ty
2928

30-
# Install other dependencies for type checking
31-
pip install fsspec
29+
# Install other dependencies for type checking
30+
pip install fsspec
3231

33-
# Run type checkers
34-
pushd ${pyarrow_dir}
35-
mypy
36-
pyright
37-
ty check;
38-
else
39-
echo "Skipping type annotation tests";
32+
# Run type checkers
33+
pushd ${pyarrow_dir}
34+
mypy
35+
pyright
36+
ty check;
4037
fi

compose.yaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -919,7 +919,6 @@ services:
919919
environment:
920920
<<: [*common, *ccache, *sccache]
921921
PYTEST_ARGS: # inherit
922-
PYARROW_TEST_ANNOTATIONS: "ON"
923922
volumes: *conda-volumes
924923
command: &python-conda-command
925924
["
@@ -1003,7 +1002,6 @@ services:
10031002
ARROW_S3: "OFF"
10041003
ARROW_SUBSTRAIT: "OFF"
10051004
ARROW_WITH_OPENTELEMETRY: "OFF"
1006-
PYARROW_TEST_ANNOTATIONS: "ON"
10071005
SETUPTOOLS_SCM_PRETEND_VERSION:
10081006
volumes: *ubuntu-volumes
10091007
deploy: *cuda-deploy
@@ -1504,7 +1502,6 @@ services:
15041502
python: ${PYTHON}
15051503
shm_size: *shm-size
15061504
environment:
1507-
PYARROW_TEST_ANNOTATIONS: "ON"
15081505
<<: [*common, *ccache, *sccache]
15091506
PARQUET_REQUIRE_ENCRYPTION: # inherit
15101507
HYPOTHESIS_PROFILE: # inherit
@@ -1536,7 +1533,6 @@ services:
15361533
BUILD_DOCS_CPP: "ON"
15371534
BUILD_DOCS_PYTHON: "ON"
15381535
PYTEST_ARGS: "--doctest-modules --doctest-cython"
1539-
PYARROW_TEST_ANNOTATIONS: "ON"
15401536
volumes: *conda-volumes
15411537
command:
15421538
["/arrow/ci/scripts/cpp_build.sh /arrow /build &&

0 commit comments

Comments
 (0)