Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: create-action-provider #470

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

feat: create-action-provider #470

wants to merge 13 commits into from
+4,358 −379

Conversation

stat
Copy link
Contributor

@stat stat commented Feb 26, 2025

What changed?

  • Documentation
  • Bug fix
  • New Action
  • New Action Provider
  • Other

Why was this change implemented?

Network support

  • All EVM
  • Base
  • Base Sepolia
  • Other

Wallet support

  • CDP Wallet
  • EVM Wallet
  • Other

Checklist

  • Changelog updated
  • Commits are signed. See instructions
  • Doc strings
  • Readme updates
  • Rebased against master
  • Relevant exports added

How has it been tested?

  • Agent tested
  • Unit tests

Notes to reviewers

@cb-heimdall
Copy link

cb-heimdall commented Feb 26, 2025 •
edited
Loading

🟡 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
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

@stat stat force-pushed the christopher.gerber/tooling/create-action-provider branch from 5eced2e to cfe2b98 Compare February 26, 2025 11:36
0xRAG
0xRAG reviewed Feb 26, 2025
View reviewed changes
@github-actions github-actions bot added documentation Improvements or additions to documentation action provider New action provider labels Feb 27, 2025
@stat stat force-pushed the christopher.gerber/tooling/create-action-provider branch from 5ede071 to 6bcaa5d Compare February 28, 2025 12:12
@github-actions github-actions bot removed the action provider New action provider label Feb 28, 2025
@github-actions github-actions bot added action provider New action provider wallet provider New wallet provider labels Feb 28, 2025
@stat stat marked this pull request as ready for review February 28, 2025 12:23
@stat stat requested a review from murrlincoln as a code owner February 28, 2025 12:23
@stat stat requested a review from CarsonRoscoe February 28, 2025 17:35
@github-actions github-actions bot removed action provider New action provider wallet provider New wallet provider labels Feb 28, 2025
@stat stat force-pushed the christopher.gerber/tooling/create-action-provider branch from 2795c34 to f45602c Compare February 28, 2025 17:42
stat
stat commented Feb 28, 2025
View reviewed changes
typescript/agentkit/package.json
@@ -17,15 +17,16 @@
"format": "prettier -c .prettierrc --write \"**/*.{ts,js,cjs,json,md}\"",
"format:check": "prettier -c .prettierrc --check \"**/*.{ts,js,cjs,json,md}\"",
"check": "tsc --noEmit",
"test": "npx jest --no-cache --testMatch='**/*.test.ts'",
"test": "npx jest --no-cache --testMatch='**/mytokenActionProvider.test.ts'",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

revert

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xRAG 0xRAG 0xRAG left review comments

@murrlincoln murrlincoln Awaiting requested review from murrlincoln murrlincoln is a code owner

@CarsonRoscoe CarsonRoscoe Awaiting requested review from CarsonRoscoe

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation needs triage python typescript
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stat @cb-heimdall @0xRAG