This is an automated email from the ASF dual-hosted git repository.
raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new fd0b94fdd9 GH-41002: [Python] Remove pins for pytest-cython and
conda-docs pytest (#45240)
fd0b94fdd9 is described below
commit fd0b94fdd9dbb0410350c9572d3df7112b40aac1
Author: Raúl Cumplido <[email protected]>
AuthorDate: Mon Jan 13 18:51:12 2025 +0100
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 <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
---
ci/conda_env_sphinx.txt | 6 +-----
docker-compose.yml | 5 -----
2 files changed, 1 insertion(+), 10 deletions(-)
diff --git a/ci/conda_env_sphinx.txt b/ci/conda_env_sphinx.txt
index 4665a32e24..751df9b2f3 100644
--- a/ci/conda_env_sphinx.txt
+++ b/ci/conda_env_sphinx.txt
@@ -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
diff --git a/docker-compose.yml b/docker-compose.yml
index 43dd3511fc..5abad2a6f8 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -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"]