kou commented on PR #33609:
URL: https://github.com/apache/arrow/pull/33609#issuecomment-1380994171
> Do I understand correctly that I should make a new issue to remove
`--disable-warnings` once the patch is included in a new release of
pytest-cython?
Sorry. I wanted to say that I want you to add a comment something like the
following:
```diff
diff --git a/docker-compose.yml b/docker-compose.yml
index 3774d55868..7cc10743e7 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -1101,7 +1101,10 @@ services:
LANG: "C.UTF-8"
BUILD_DOCS_CPP: "ON"
BUILD_DOCS_PYTHON: "ON"
- PYTEST_ARGS: "--doctest-modules --doctest-cython"
+ # GH-31506/GH-33609: Remove --disable-warnings once
+ # https://github.com/lgpage/pytest-cython/issues/24 is resolved
+ # and released a new version that includes the fix.
+ PYTEST_ARGS: "--doctest-modules --doctest-cython --disable-warnings"
volumes: *conda-volumes
command:
["/arrow/ci/scripts/cpp_build.sh /arrow /build &&
```
But creating a new issue for the removal as you said is a good idea!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
