Merge pull request #1992 from PrefectHQ/remove-airflow

Remove airflow tasks

.circleci/config.yml

@@ -238,37 +238,6 @@ jobs:
           name: Run tests
           command: pytest -vvrfEsx .
 
-  test_airflow:
-    docker:
-      - image: continuumio/miniconda3:4.6.14
-    steps:
-      - checkout
-      - setup_remote_docker
-      - run:
-          name: Install gcc
-          command: apt-get update && apt-get install -y gcc
-
-      - run:
-          name: Create Airflow conda environment
-          command: conda create -n airflow python=3.6 pip -y
-
-      - run:
-          name: Install Airflow
-          command: source activate airflow && SLUGIFY_USES_TEXT_UNIDECODE=yes pip install apache-airflow flask==1.0.4 && source deactivate
-
-      - run:
-          name: Install prefect
-          command: pip install ".[all_extras]"
-
-      - run:
-          name: Run tests w/ airflow
-          command: pytest -vvrfEsx --airflow --cov=prefect --cov-report=xml:/tmp/workspace/coverage/airflow-coverage.xml .
-
-      - persist_to_workspace:
-          root: *workspace_root
-          paths:
-            - coverage
-
   upload_coverage:
     docker:
       - image: python:3.6
@@ -393,13 +362,11 @@ workflows:
       - test_lower_prefect
       - test_vanilla_prefect
       - test_py352_import_prefect
-      - test_airflow
       - upload_coverage:
           requires:
             - test_35
             - test_36
             - test_vanilla_prefect
-            - test_airflow
 
   "Check code style and docs":
     jobs:

CHANGELOG.md

@@ -26,7 +26,7 @@ These changes are available in the [master branch](https://github.com/PrefectHQ/
 
 ### Breaking Changes
 
-- None
+- Remove Airflow Tasks - [#1992](https://github.com/PrefectHQ/prefect/pull/1992)
 
 ### Contributors
 

conftest.py

@@ -4,12 +4,6 @@
 
 
 def pytest_addoption(parser):
-    parser.addoption(
-        "--airflow",
-        action="store_true",
-        dest="airflow",
-        help="including this flag will attempt to ONLY run airflow compatibility tests",
-    )
     parser.addoption(
         "--skip-formatting",
         action="store_true",
@@ -19,28 +13,13 @@ def pytest_addoption(parser):
 
 
 def pytest_configure(config):
-    config.addinivalue_line(
-        "markers", "airflow: mark test to run only when --airflow flag is provided."
-    )
     config.addinivalue_line(
         "markers",
         "formatting: mark test as formatting to skip when --skip-formatting flag is provided.",
     )
 
 
 def pytest_runtest_setup(item):
-    air_mark = item.get_closest_marker("airflow")
-
-    # if a test IS marked as "airflow" and the airflow flag IS NOT set, skip it
-    if air_mark is not None and item.config.getoption("--airflow") is False:
-        pytest.skip(
-            "Airflow tests skipped by default unless --airflow flag provided to pytest."
-        )
-
-    # if a test IS NOT marked as airflow and the airflow flag IS set, skip it
-    elif air_mark is None and item.config.getoption("--airflow") is True:
-        pytest.skip("Non-Airflow tests skipped because --airflow flag was provided.")
-
     formatting_mark = item.get_closest_marker("formatting")
 
     # if a test IS marked as "formatting" and the --skip-formatting flag IS set, skip it

docs/outline.toml

@@ -180,11 +180,6 @@ module = "prefect.tasks.control_flow"
 classes = ["FilterTask"]
 functions = ["switch", "ifelse", "merge"]
 
-[pages.tasks.airflow]
-title = "Airflow Tasks"
-module = "prefect.tasks.airflow"
-classes = ["AirflowTask", "AirflowTriggerDAG"]
-
 [pages.tasks.airtable]
 title = "Airtable Tasks"
 module = "prefect.tasks.airtable"