Skip to content

Commit e92f030

Browse files
committed
refactor: Make apache-airflow not required
Since we wish to support all MWAA supported versions of Airflow, this requires supporting both 1.10.12 and 2.X releases. However, There are several conflicting dependencies between dbt-core and apache-airflow. None of the conflicts should break us, but they do make dependency resolution impossible, which means airflow-dbt-python cannot be installed. For this reason we have removed the dependency to apache-airflow. Assuming that as an Airflow operator we are always installing airflow-dbt-python in an environment that already has Airflow installed, this shouldn't cause any issues in production deployments. Moreover, this makes testing multiple Airflow versions easier as we can pre-install apache-airflow.
1 parent b1b2b3a commit e92f030

File tree

3 files changed

+303
-225
lines changed

3 files changed

+303
-225
lines changed

.github/workflows/test.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
with:
2727
poetry-version: 1.1.12
2828
- name: Install airflow-dbt-python with Poetry
29-
run: poetry install -E amazon -E postgres
29+
run: poetry install -E amazon -E postgres -E airflow
3030
- name: Style guide enforcement with flake8
3131
run: poetry run flake8 .
3232
- name: Static type checking with mypy

0 commit comments

Comments
 (0)