Skip to content

chore: rename scripts/test-branch-on-hetzner to test-on-hetzner - #753

Merged
brawer merged 2 commits into
mainfrom
rename-test-branch-on-hetzner
Aug 23, 2026
Merged

chore: rename scripts/test-branch-on-hetzner to test-on-hetzner#753
brawer merged 2 commits into
mainfrom
rename-test-branch-on-hetzner

Conversation

@brawer

@brawer brawer commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

What

Pure rename: scripts/test-branch-on-hetzner/scripts/test-on-hetzner/, plus updating every reference to the old path.

Why

It's not just for testing a branch anymore. deploy --image/--branch, start --containerized, validate, and the bucket lifecycle all work identically whether the container was built from a dev branch or pulled as an already-built release image -- verified end-to-end today: the #749 investigation ran the fix straight from main, and the very next smoke test used a real v0.8.0 release image, through the exact same containerized path. test-branch-on-hetzner undersold that.

Scope

git mv (preserves history) plus reference updates in scripts/README.md, scripts/test-branch-on-macos/{README.md,test_macos.py}, docs/TESTING.md, .github/workflows/{test-scripts,shellcheck}.yml (comments only -- their actual path-filter triggers are scripts/**/**/*.sh globs, not hardcoded to the old directory name, so no trigger changes needed), and src/pipeline/osm/fetch.rs's one comment. Also tightened the tool's own opening lines (README.md's title, cloud_test.py's module docstring), which still described it as branch-only.

No logic changes. uv run pytest (78/78), cargo build, and uvx ruff check (same 2 pre-existing findings as before) all clean from the new location.

🤖 Generated with Claude Code

@brawer
brawer force-pushed the rename-test-branch-on-hetzner branch 3 times, most recently from 5290065 to 72a97a7 Compare August 23, 2026 07:39
It's not just for testing a branch anymore -- deploy --image/--branch,
start --containerized, validate, and the bucket lifecycle all work
identically whether the container was built from a dev branch or
pulled as an already-built release image (verified end-to-end: the
#749 investigation ran the fix straight from main, and the very next
smoke test used a real v0.8.0 release image, through the exact same
containerized path). 'test-branch-on-hetzner' undersold that for a
while.

Pure rename plus reference updates (README.md/cloud_test.py's own
opening lines included, since those undersold it too) -- no logic
changes. git mv preserved history; CI workflow triggers didn't need
touching (scripts/**, **/*.sh globs, not hardcoded to the old path).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@brawer
brawer force-pushed the rename-test-branch-on-hetzner branch from 72a97a7 to e7190c6 Compare August 23, 2026 07:40
@brawer
brawer enabled auto-merge August 23, 2026 07:42
@brawer
brawer added this pull request to the merge queue Aug 23, 2026
Merged via the queue into main with commit abef5cf Aug 23, 2026
6 checks passed
@brawer
brawer deleted the rename-test-branch-on-hetzner branch August 23, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant