Skip to content

Commit

Permalink
GH-41002: [Python] Remove pins for pytest-cython and conda-docs pytest (
Browse files Browse the repository at this point in the history
#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 <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
  • Loading branch information
raulcd authored Jan 13, 2025
1 parent 32fcd18 commit fd0b94f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 10 deletions.
6 changes: 1 addition & 5 deletions ci/conda_env_sphinx.txt
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,5 @@ sphinx-lint
sphinxcontrib-jquery
sphinxcontrib-mermaid
sphinx==6.2
# Requirement for doctest-cython
# Needs upper pin of 0.3.0, see:
# https://github.com/lgpage/pytest-cython/issues/67
# With 0.3.* bug fix release, the pin can be removed
pytest-cython==0.2.2
pytest-cython
pandas
5 changes: 0 additions & 5 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -1429,15 +1429,10 @@ services:
BUILD_DOCS_PYTHON: "ON"
PYTEST_ARGS: "--doctest-modules --doctest-cython"
volumes: *conda-volumes
# pytest is installed with an upper pin of 8.0.0 because
# newer version breaks cython doctesting, see:
# https://github.com/lgpage/pytest-cython/issues/58
# Remove pip install pytest~=7 when upstream issue is resolved
command:
["/arrow/ci/scripts/cpp_build.sh /arrow /build &&
/arrow/ci/scripts/python_build.sh /arrow /build &&
pip install -e /arrow/dev/archery[numpydoc] &&
pip install pytest~=7.4 &&
archery numpydoc --allow-rule GL10,PR01,PR03,PR04,PR05,PR10,RT03,YD01 &&
/arrow/ci/scripts/python_test.sh /arrow"]

Expand Down

0 comments on commit fd0b94f

Please sign in to comment.