From df948207c6869a6eeb09d66069bf442c57474a25 Mon Sep 17 00:00:00 2001 From: Jammy2211 Date: Thu, 23 Jul 2026 17:37:57 +0100 Subject: [PATCH] chore: rename env profile to profile_smoke.yaml (#161 step 6) config/build/env_vars.yaml -> profile_smoke.yaml (pure rename; no release profile in this repo). Vendored run_smoke.py prefers the canonical name with a legacy fallback until the stage-3 cleanup. Repo-local references updated. Tooling accepts both names since PyAutoHands#183 / PyAutoHeart#101. Co-Authored-By: Claude Fable 5 --- .github/scripts/run_smoke.py | 7 +++++-- AGENTS.md | 2 +- config/build/{env_vars.yaml => profile_smoke.yaml} | 2 +- 3 files changed, 7 insertions(+), 4 deletions(-) rename config/build/{env_vars.yaml => profile_smoke.yaml} (96%) diff --git a/.github/scripts/run_smoke.py b/.github/scripts/run_smoke.py index 3822a45..e13f3fb 100644 --- a/.github/scripts/run_smoke.py +++ b/.github/scripts/run_smoke.py @@ -1,7 +1,7 @@ """ Run the workspace smoke test suite. -Reads `smoke_tests.txt` from the workspace root and `config/build/env_vars.yaml` +Reads `smoke_tests.txt` from the workspace root and `config/build/profile_smoke.yaml` for per-script env var overrides, then runs each listed script with the appropriate environment. Continues through failures and exits non-zero if any script failed. @@ -23,7 +23,10 @@ WORKSPACE = Path(__file__).resolve().parents[2] SMOKE_FILE = WORKSPACE / "smoke_tests.txt" -ENV_VARS_FILE = WORKSPACE / "config" / "build" / "env_vars.yaml" +ENV_VARS_FILE = WORKSPACE / "config" / "build" / "profile_smoke.yaml" +if not ENV_VARS_FILE.exists(): + # Legacy fallback; removed at the stage-3 cleanup once env_vars.yaml is gone. + ENV_VARS_FILE = WORKSPACE / "config" / "build" / "env_vars.yaml" SCRIPTS_DIR = WORKSPACE / "scripts" diff --git a/AGENTS.md b/AGENTS.md index d7e8b22..5ca2bef 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -38,7 +38,7 @@ cap grids/masks). ## Testing On CI, every PR is gated on Python **3.12 and 3.13** by `smoke_tests.yml` (runs -`python .github/scripts/run_smoke.py`, driven by `smoke_tests.txt` + `config/build/env_vars.yaml` — +`python .github/scripts/run_smoke.py`, driven by `smoke_tests.txt` + `config/build/profile_smoke.yaml` — the definition of green), `navigator_check.yml` (PyAutoHands's reusable navigator-catalogue check; see *Notebooks vs Scripts*), and `url_check.yml` (link checking). The smoke and navigator jobs check out **PyAutoHands** as a sibling and run the PyAuto* libraries from the **same-named branch** of each diff --git a/config/build/env_vars.yaml b/config/build/profile_smoke.yaml similarity index 96% rename from config/build/env_vars.yaml rename to config/build/profile_smoke.yaml index 7e39cc4..2bcf776 100644 --- a/config/build/env_vars.yaml +++ b/config/build/profile_smoke.yaml @@ -28,7 +28,7 @@ defaults: # 2026-07-23: the remaining five overrides were removed for the same reason — # `imaging/start_here`, `interferometer/start_here`, `group/start_here`, # `multi/start_here` and `guides/results/` are all autolens_workspace paths -# copied in from that repo's env_vars.yaml. HowToLens contains only +# copied in from that repo's profile_smoke.yaml. HowToLens contains only # chapter_*/tutorial_*.py, so none of them ever matched a file here. # Do not copy override patterns between repos; a pattern that matches nothing # is silently inert, not an override. The list is now empty, and `overrides: []`