Skip to content

Point the dev-channel checkout clone at omacom/omarchy - #9088

Open
PixelatedContinuum wants to merge 1 commit into
omacom:quattrofrom
PixelatedContinuum:fix/dev-channel-clone-url
Open

Point the dev-channel checkout clone at omacom/omarchy#9088
PixelatedContinuum wants to merge 1 commit into
omacom:quattrofrom
PixelatedContinuum:fix/dev-channel-clone-url

Conversation

@PixelatedContinuum

Copy link
Copy Markdown

Same rename as #9080 and #9087 (basecamp/omarchyomacom/omarchy), split
out separately because this one is coupled to a test rather than a plain
reference swap.

bin/omarchy-channel-set's dev channel clones the source repo directly to
~/omarchy:

[[ -d $checkout/.git ]] || git clone https://github.com/basecamp/omarchy.git "$checkout"

test/shell.d/channel-test.sh asserts that exact clone command, including the
URL, verbatim — so fixing the script without the test would fail CI, and
fixing the test without the script would just pin the stale URL harder. Both
move together in this commit.

Verified before changing anything

git ls-remote https://github.com/basecamp/omarchy.git HEAD resolves
correctly today (GitHub's git smart-HTTP protocol follows the org rename), so
this isn't fixing a live clone failure — it's the same future-proofing as the
other two PRs.

Testing

bash test/shell.d/channel-test.sh — all 33 assertions pass, including "dev
clones the source checkout to ~/omarchy" and "dev activates the checkout
before changing or updating packages," against the updated URL.

./test/all — same 5 pre-existing, unrelated failures as a clean quattro
checkout (see #9087 for detail); channel-test.sh is not among them.

bin/omarchy-channel-set's 'dev' channel clones the source repo directly
to ~/omarchy. Its URL still used the pre-rename basecamp/omarchy org.
Updated test/shell.d/channel-test.sh in the same commit since it asserts
that clone URL verbatim; test/shell.d/channel-test.sh passes against the
updated command.
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