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.

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

Add Palm and Palm Testnet chains #1691

Merged
merged 6 commits into from
Jan 15, 2024
Merged

Add Palm and Palm Testnet chains #1691

merged 6 commits into from
Jan 15, 2024

Conversation

skenaja
Copy link
Contributor

@skenaja skenaja commented Jan 15, 2024

PR-Codex overview

Focus of the PR:

This PR focuses on adding support for the Palm and Palm Testnet chains.

Detailed summary:

  • Added the palm chain definition.
  • Added the palmTestnet chain definition.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented Jan 15, 2024

🦋 Changeset detected

Latest commit: 4aaa40b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Jan 15, 2024

@skenaja is attempting to deploy a commit to the wagmi Team on Vercel.

A member of the Team first needs to authorize it.

src/chains/definitions/palm.ts Outdated Show resolved Hide resolved
src/chains/definitions/palmTestnet.ts Outdated Show resolved Hide resolved
src/chains/definitions/palm.ts Outdated Show resolved Hide resolved
src/chains/definitions/palmTestnet.ts Outdated Show resolved Hide resolved
skenaja and others added 4 commits January 15, 2024 17:17
Co-authored-by: awkweb <tom@meagher.co>
Co-authored-by: awkweb <tom@meagher.co>
Co-authored-by: awkweb <tom@meagher.co>
Co-authored-by: awkweb <tom@meagher.co>
@skenaja
Copy link
Contributor Author

skenaja commented Jan 15, 2024

Hi @tmm - suggested changes were applied.

FYI I was following this guide when adding infura RPCs: https://github.com/wevm/viem/blob/main/.github/CONTRIBUTING.md?plain=1#L210

@tmm tmm merged commit 8d011e8 into wevm:main Jan 15, 2024
2 of 3 checks passed
@github-actions github-actions bot mentioned this pull request Jan 15, 2024
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.

None yet

2 participants