Remove legacy Airflow 3 test tooling#2346
Merged
Merged
Conversation
✅ Deploy Preview for astronomer-cosmos canceled.
|
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the legacy scripts/airflow3/ developer tooling for local Airflow 3 setups, aligning Airflow 3 testing with the documented hatch test matrix approach.
Changes:
- Deleted Airflow 3 local setup scripts (env setup, venv creation, install-from-main workflow).
- Removed the Airflow 3 pinned requirements list and local test helper script.
- Removed related README and
.gitignoreentries underscripts/airflow3/.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/airflow3/tests.sh | Removed legacy Airflow 3 test helper script. |
| scripts/airflow3/setup.sh | Removed legacy venv creation and dependency install script for Airflow 3. |
| scripts/airflow3/requirements.txt | Removed legacy Airflow 3 requirements list. |
| scripts/airflow3/install_from_main.sh | Removed legacy script for installing Airflow from main branch. |
| scripts/airflow3/env.sh | Removed legacy environment variable setup for local Airflow 3. |
| scripts/airflow3/README.md | Removed legacy documentation for running Airflow 3 locally. |
| scripts/airflow3/.gitignore | Removed ignore rules specific to the deleted Airflow 3 tooling. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
pankajkoti
force-pushed
the
cleanup-airflow3-tooling
branch
from
3a34cdc to
3af5110
Compare
tatiana
approved these changes
Feb 11, 2026
3 tasks
pankajastro
added a commit
that referenced
this pull request
May 11, 2026
## Summary - Follow-up to #2346, which removed the Airflow 3 dev tooling under `scripts/airflow3/` (`setup.sh`, `env.sh`, `install_from_main.sh`, `requirements.txt`, `tests.sh`, `README.md`, `.gitignore`) and redirected users to the hatch test matrix. That PR missed the `dags/` subfolder, leaving 8 files behind. - Nothing in the repo references `scripts/airflow3/` anymore (verified via `grep` across `*.sh`, `*.py`, `*.yaml`, `*.yml`, `*.toml`, `*.cfg`, `Dockerfile*`, `*.md`, `*.rst` — the only hit is an obsolete CHANGELOG entry). - Of the 8 remaining files: 5 are symlinks back to `dev/dags/`, 2 (`basic_cosmos_dag.py`, `example_operators.py`) are stale/diverged copies of files that already exist with up-to-date content in `dev/dags/`, and `hello.py` is a 29-line BashOperator hello-world smoke test with no Cosmos imports. Deletes the directory to finish the cleanup started in #2346. ## Test plan - [x] `git grep` confirms no remaining references to `scripts/airflow3` in code, scripts, CI, or docs - [x] Existing `dev/dags/` contains current, maintained versions of the non-trivial DAGs (`basic_cosmos_dag.py`, `basic_cosmos_task_group.py`, `example_operators.py`, `example_virtualenv*.py`, `jaffle_shop_kubernetes.py`, `simple_dag_async.py`) - [x] No CI workflow or hatch script invokes anything from this path 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
tatiana
pushed a commit
that referenced
this pull request
May 20, 2026
## Summary - Follow-up to #2346, which removed the Airflow 3 dev tooling under `scripts/airflow3/` (`setup.sh`, `env.sh`, `install_from_main.sh`, `requirements.txt`, `tests.sh`, `README.md`, `.gitignore`) and redirected users to the hatch test matrix. That PR missed the `dags/` subfolder, leaving 8 files behind. - Nothing in the repo references `scripts/airflow3/` anymore (verified via `grep` across `*.sh`, `*.py`, `*.yaml`, `*.yml`, `*.toml`, `*.cfg`, `Dockerfile*`, `*.md`, `*.rst` — the only hit is an obsolete CHANGELOG entry). - Of the 8 remaining files: 5 are symlinks back to `dev/dags/`, 2 (`basic_cosmos_dag.py`, `example_operators.py`) are stale/diverged copies of files that already exist with up-to-date content in `dev/dags/`, and `hello.py` is a 29-line BashOperator hello-world smoke test with no Cosmos imports. Deletes the directory to finish the cleanup started in #2346. ## Test plan - [x] `git grep` confirms no remaining references to `scripts/airflow3` in code, scripts, CI, or docs - [x] Existing `dev/dags/` contains current, maintained versions of the non-trivial DAGs (`basic_cosmos_dag.py`, `basic_cosmos_task_group.py`, `example_operators.py`, `example_virtualenv*.py`, `jaffle_shop_kubernetes.py`, `simple_dag_async.py`) - [x] No CI workflow or hatch script invokes anything from this path 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR removes the
scripts/airflow3/dev tooling added in PR #1627 (venv, env vars, install-from-main script, etc.).The recommendation to test Airflow 3 is to use hatch test matrix (e.g. hatch run tests:test with matrix airflow=3.0 or 3.1) to run Cosmos with Airflow 3.
closes: #1734
related: #1627