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 Shape chain #2764

Merged
merged 10 commits into from
Sep 25, 2024
Merged

add Shape chain #2764

merged 10 commits into from
Sep 25, 2024

Conversation

j6i
Copy link
Contributor

@j6i j6i commented Sep 24, 2024

adds the Shape chain definition


PR-Codex overview

This PR introduces a new blockchain chain named Shape, adding its configuration and exporting it for use in the application.

Detailed summary

  • Added a new chain definition for Shape in src/chains/definitions/shape.ts.
  • Configured Shape with properties like id, name, nativeCurrency, rpcUrls, and blockExplorers.
  • Exported the shape chain from src/chains/index.ts and src/op-stack/chains.ts.

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

Copy link

vercel bot commented Sep 24, 2024

@j6i is attempting to deploy a commit to the Wevm Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

changeset-bot bot commented Sep 24, 2024

🦋 Changeset detected

Latest commit: 7c8c4c0

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

@j6i j6i closed this Sep 24, 2024
@j6i j6i reopened this Sep 24, 2024
@j6i
Copy link
Contributor Author

j6i commented Sep 24, 2024

Gm @jxom, please review when you get a chance

...chainConfig.contracts,
l2OutputOracle: {
[sourceId]: {
address: '0x6Ef8c69CfE4635d866e3E02732068022c06e724D',
Copy link
Member

Choose a reason for hiding this comment

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

Where are these addresses documented?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jxom jxom merged commit e0a6316 into wevm:main Sep 25, 2024
9 of 17 checks passed
@github-actions github-actions bot mentioned this pull request Sep 25, 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.

2 participants