potiuk commented on a change in pull request #20913:
URL: https://github.com/apache/airflow/pull/20913#discussion_r790887547
##########
File path: dev/breeze/setup.cfg
##########
@@ -66,6 +67,7 @@ where=src
console_scripts=
Breeze2=airflow_breeze.breeze:main
airflow-freespace=airflow_ci.freespace:main
+
test-examples-of-prod-image-building=airflow_ci.test_examples_of_prod_image_building:main
Review comment:
Yeah. It's not needed really, because we only run the tests as "pytest"
tests. In case of the "airflow-freespace" that was a "standalone" python code
that we had to turn into entrypoint - but in this case we really want to just
run pytest tests and `pytest` is an entrypoint on its own.
So our "entrypoint" is "pytest" and our "test_*.py" is simply parameter
passed to pytest to tell it what it should run. The `python -m pytest` is just
recommended way of running `pytest` entrypoint:
https://docs.pytest.org/en/6.2.x/usage.html#calling-pytest-through-python-m-pytest
- we could also do it via `pytest` (and there it will be similar to
`airlfow-freespace` but i prefer to use `python -m` wherever possible as it is
more portable.
--
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]
