samredai commented on code in PR #4792:
URL: https://github.com/apache/iceberg/pull/4792#discussion_r885200858
##########
.github/workflows/python-ci.yml:
##########
@@ -50,5 +49,7 @@ jobs:
run: |
pip install -e .[dev]
pip install -U tox-gh-actions
+ - name: Start MinIO instance
+ run: docker-compose -f python/docker-compose.ci.yml up -d
- working-directory: ./python
- run: tox
+ run: tox -- -m "marked or not marked"
Review Comment:
I don't think we want the tests marked `s3` to run by default locally, so I
added `-m "not s3"` in the tox.ini file. Here what I'm doing is overriding that
setting for the CI.
I couldn't find a `-m` argument dedicated for directing pytest to run
**all** tests, including unmarked tests, so I do a bit of a trick here:
although there is no group of tests named `marked`, telling pytest to run
"marked or not marked" results in all tests running.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
