Skip to content

chore: remove Base Account and migration content - #162

Merged
youssefea merged 3 commits into
masterfrom
youssef/remove-base-account
Sep 17, 2026
Merged

youssefea merged 3 commits into
masterfrom
youssef/remove-base-account

Conversation

@youssefea

@youssefea youssefea commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • remove the Base Account section and supporting references from build-on-base
  • remove all migration references and migration-only scripts from build-on-base
  • remove remaining Base Account mentions from repository docs

Validation

  • quick_validate.py skills/build-on-base
  • git diff --check

Co-authored-by: Codex <codex-noreply@coinbase.com>
@cb-heimdall

cb-heimdall commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@youssefea youssefea changed the title chore: remove Base Account content chore: remove Base Account + OCK content Sep 16, 2026
Co-authored-by: Codex <codex-noreply@coinbase.com>
@youssefea youssefea changed the title chore: remove Base Account + OCK content chore: remove Base Account and OnchainKit migration content Sep 16, 2026
Co-authored-by: Codex <codex-noreply@coinbase.com>
@youssefea youssefea changed the title chore: remove Base Account and OnchainKit migration content chore: remove Base Account and migration content Sep 16, 2026
@youssefea
youssefea merged commit 15adb8d into master Sep 17, 2026
4 checks passed
erhnysr added a commit to erhnysr/skills that referenced this pull request Oct 12, 2026
master removed the MiniKit-to-Farcaster migration content in base#162, so
this branch's link fix in that mapping.md is dropped with the file. The
--broadcast and API-dashboard fixes in deploy-contracts.md and the
wallet link in agents/register.md are unchanged.
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.

3 participants