Skip to content

Comments

Fix README instructions, update links and Node.js references in examples#964

Merged
phdargen merged 13 commits intocoinbase:mainfrom
beck042:path&&linksLICENSE
Feb 24, 2026
Merged

Fix README instructions, update links and Node.js references in examples#964
phdargen merged 13 commits intocoinbase:mainfrom
beck042:path&&linksLICENSE

Conversation

@beck042
Copy link
Contributor

@beck042 beck042 commented Feb 22, 2026

Description

This PR includes several improvements and fixes across the TypeScript examples:

  1. Correct non-existent directory in README

    • Replaced typescript/examples/langchain-cdp-v2-chatbot with typescript/examples/langchain-cdp-chatbot in README.md.
    • Removed references to langchain-cdp-v2-chatbot in Contributing.md.
  2. Add proper license links

    • Updated all example READMEs to link to the LICENSE file (../../../LICENSE.md) instead of just stating "Apache-2.0".
  3. Update Node.js references

    • Replaced plain text Node.js 18+ with a clickable link: [Node.js 18+](https://nodejs.org/en/download/current).

These changes improve clarity for users following setup instructions, ensure links are accurate, and standardize license references across examples.

Checklist

A couple of things to include in your PR for completeness:

  • Added documentation to all relevant README.md files
  • Added a changelog entry

@beck042 beck042 requested a review from murrlincoln as a code owner February 22, 2026 09:22
@cb-heimdall
Copy link

cb-heimdall commented Feb 22, 2026

✅ 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

@github-actions github-actions bot added documentation Improvements or additions to documentation example New example agent labels Feb 22, 2026
@phdargen phdargen self-assigned this Feb 24, 2026
Copy link
Contributor

@phdargen phdargen left a comment

Choose a reason for hiding this comment

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

Thanks for the fixes @beck042

@phdargen phdargen merged commit c1cf714 into coinbase:main Feb 24, 2026
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation example New example agent

Development

Successfully merging this pull request may close these issues.

3 participants