Skip to content

migrate.sh: auto-clone + inline gather/extract bash in publish.yml#36

Merged
coretl merged 1 commit into
mainfrom
refactor-gather-priority
Jun 30, 2026
Merged

migrate.sh: auto-clone + inline gather/extract bash in publish.yml#36
coretl merged 1 commit into
mainfrom
refactor-gather-priority

Conversation

@coretl

@coretl coretl commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • publish.yml: inline the gather and extract steps as bash (removing the separate assemble.sh), and fix runner.temp being unavailable in reusable workflow job-level env vars
  • migrate.sh: auto-clone the target repo when not already running inside it — the script now detects its origin URL and clones to a temp dir if needed, so operators can run it directly from the myst-version-switcher-plugin devcontainer without a separate clone step
  • Docs: update migrate how-to to say run from the devcontainer, add gh auth login as an explicit prerequisite, remove the "clone the repo first" requirement

Test plan

  • Verify publish.yml gather + extract steps work on a PR and main push (inline bash replaces assemble.sh)
  • Run scripts/migrate.sh ORG/REPO --dry-run from the devcontainer (not inside the target repo) and confirm it clones automatically
  • Run the same from inside a clone of the target repo and confirm it uses the existing clone (no re-clone)
  • Check that EXIT trap cleans up the temp dir after a dry-run

🤖 Generated with Claude Code

Operators previously had to clone the target repo and run the script
from within it. The script now detects whether it is already running
inside a clone of the target repo (by comparing the normalised origin
URL); if not, it clones to a temp directory and cd's into it, cleaning
up on exit. Docs updated to say to run from the devcontainer and to
include gh auth login as a prerequisite.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coretl coretl merged commit 1f80de7 into main Jun 30, 2026
7 checks passed
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