Jarek Potiuk
Jarek Potiuk
(The error is also present in all other 64.* and 65* versions BTW).
> By having a look on the [`setup.py`](https://github.com/apache/airflow/blob/98a7701942c683f3126f9c4f450c352b510a2734/setup.py#L826), it seems that `airflow` implements a custom `develop` command. Thanks for the pointer @abravalheri . I will take a closer look and...
Time to do some more deep PEP reading
> @potiuk Hey! Airflow's `setup.py` looks quite complex which intrigues me. Would you be interested in me opening a PR for a PoC switch to Hatchling (which will fix your...
Very nice change! Since it's learning exercise I tried to be thorough reviewer and pointed out some minute details.
> cc @potiuk regarding if this would work in Airflow Generally yes. There are of course some details - particularly about optionality of those providers. 1. What we really would...
> Could you also update the constraint file ? Constraints are updated only when they prevent things from being build. We never update constraints with new providers - they are...
The failing integration test case is a flake - mssql does not have enough time to start. This has been fixed in #39866
> I ran `breeze testing tests --upgrade-boto tests/providers/docker` in main branch with latest CI image - getting same error results. https://github.com/apache/airflow/pull/39747 should fix the docker test in "Latest Botocore".
As discussed with @mobuchowski -> we need to add back-compatibility `try/import` to past versions of the openlineage provider. We could even potentially add back-compatibility test suite for older version of...