Skip to content

remove legacy examples#1356

Open
phdargen wants to merge 2 commits intocoinbase:mainfrom
phdargen:remove-legacy-examples
Open

remove legacy examples#1356
phdargen wants to merge 2 commits intocoinbase:mainfrom
phdargen:remove-legacy-examples

Conversation

@phdargen
Copy link
Copy Markdown
Contributor

Description

Removed all legacy examples

Tests

Checklist

  • I have formatted and linted my code
  • All new and existing tests pass
  • My commits are signed (required for merge) -- you may need to rebase if you initially pushed unsigned commits
  • I added a changelog fragment for user-facing changes (docs-only changes can skip)

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 26, 2026

@phdargen is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

@cb-heimdall
Copy link
Copy Markdown

cb-heimdall commented Feb 26, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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

@github-actions github-actions bot added examples Changes to examples legacy Changes to legacy sdk or examples labels Feb 26, 2026
@phdargen phdargen force-pushed the remove-legacy-examples branch 3 times, most recently from ff49f11 to c3fabb0 Compare March 6, 2026 04:54
Copy link
Copy Markdown
Contributor

@CarsonRoscoe CarsonRoscoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Once #1544 (review) is released and a new tag for latest legacy state is ready, can add to /examples/python/README.md and /examples/typescript/README.md a section at the bottom called "Legacy Examples" which links the tag to view the removed legacy examples?

@phdargen phdargen force-pushed the remove-legacy-examples branch from c3fabb0 to 2693498 Compare March 30, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

examples Changes to examples legacy Changes to legacy sdk or examples

Development

Successfully merging this pull request may close these issues.

4 participants