Skip to content

Commit fd0b94f

Browse files
authored
GH-41002: [Python] Remove pins for pytest-cython and conda-docs pytest (#45240)
### Rationale for this change pytest-cython 0.3.1 was releases with the fix required for our tests to pass. ### What changes are included in this PR? Remove unnecessary dependency pinned versions. ### Are these changes tested? Via existing CI ### Are there any user-facing changes? No * GitHub Issue: #45239 * GitHub Issue: #41002 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
1 parent 32fcd18 commit fd0b94f

File tree

2 files changed

+1
-10
lines changed

2 files changed

+1
-10
lines changed

ci/conda_env_sphinx.txt

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,5 @@ sphinx-lint
3030
sphinxcontrib-jquery
3131
sphinxcontrib-mermaid
3232
sphinx==6.2
33-
# Requirement for doctest-cython
34-
# Needs upper pin of 0.3.0, see:
35-
# https://github.com/lgpage/pytest-cython/issues/67
36-
# With 0.3.* bug fix release, the pin can be removed
37-
pytest-cython==0.2.2
33+
pytest-cython
3834
pandas

docker-compose.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1429,15 +1429,10 @@ services:
14291429
BUILD_DOCS_PYTHON: "ON"
14301430
PYTEST_ARGS: "--doctest-modules --doctest-cython"
14311431
volumes: *conda-volumes
1432-
# pytest is installed with an upper pin of 8.0.0 because
1433-
# newer version breaks cython doctesting, see:
1434-
# https://github.com/lgpage/pytest-cython/issues/58
1435-
# Remove pip install pytest~=7 when upstream issue is resolved
14361432
command:
14371433
["/arrow/ci/scripts/cpp_build.sh /arrow /build &&
14381434
/arrow/ci/scripts/python_build.sh /arrow /build &&
14391435
pip install -e /arrow/dev/archery[numpydoc] &&
1440-
pip install pytest~=7.4 &&
14411436
archery numpydoc --allow-rule GL10,PR01,PR03,PR04,PR05,PR10,RT03,YD01 &&
14421437
/arrow/ci/scripts/python_test.sh /arrow"]
14431438

0 commit comments

Comments
 (0)