Skip to content

Drop temporary android-sample clone pin#1

Open
phwizard wants to merge 1 commit into
mainfrom
tf/unpin-android-sample
Open

Drop temporary android-sample clone pin#1
phwizard wants to merge 1 commit into
mainfrom
tf/unpin-android-sample

Conversation

@phwizard
Copy link
Copy Markdown
Member

Summary

  • The temp pin (commit be73aa0) pointed ethora setup at the merged tf/setup-integration-and-ux-fixes branch in ethora-sample-android.
  • That branch was merged into sample-android main on 2026-05-08, so the pin now clones a frozen pre-merge state and the CLI's .env keys are once again actively read by main.
  • Removes the branch: override on the android SDK_REPOS entry. Other targets (swift, reactjs, reactnative, wordpress) already use default-branch clones via the same optional branch? field.

Test plan

  • npm run build clean (done)
  • ethora setup → pick Android target → confirm clone command is git clone https://github.com/dappros/ethora-sample-android.git ... with no -b flag
  • Sample boots with .env from setup (BuildConfig wiring works on main)

sample-android PR #1 (tf/setup-integration-and-ux-fixes) merged on
2026-05-08, so the pin now clones a frozen pre-merge state. The sample's
Setup-tab → BuildConfig wiring + .env policy that this pin was protecting
is on main now, so default-branch clone is correct again.

Other SDK targets (swift, reactjs, reactnative, wordpress) already use
the default branch via the optional `branch?` field — this just removes
the android-specific override.
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