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

feat: add base rpcUrls for Alchemy #1175

Merged
merged 3 commits into from
Sep 10, 2023
Merged

feat: add base rpcUrls for Alchemy #1175

merged 3 commits into from
Sep 10, 2023

Conversation

avasisht23
Copy link
Contributor

@avasisht23 avasisht23 commented Sep 7, 2023

PR-Codex overview

This PR focuses on adding Alchemy RPC URLs for the Base chain.

Detailed summary

  • Added Alchemy RPC URLs for the Base chain in src/chains/definitions/base.ts.
  • Added Alchemy RPC URLs for the Base Goerli chain in src/chains/definitions/baseGoerli.ts.

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

@vercel
Copy link

vercel bot commented Sep 7, 2023

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

A member of the Team first needs to authorize it.

@changeset-bot
Copy link

changeset-bot bot commented Sep 7, 2023

🦋 Changeset detected

Latest commit: 629697e

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

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

@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -8.20% ⚠️

Comparison is base (2b64f70) 99.80% compared to head (4dfe334) 91.61%.
Report is 3 commits behind head on main.

❗ Current head 4dfe334 differs from pull request most recent head 629697e. Consider uploading reports for the commit 629697e to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1175      +/-   ##
==========================================
- Coverage   99.80%   91.61%   -8.20%     
==========================================
  Files         399      390       -9     
  Lines       31313    30462     -851     
  Branches     1919     1411     -508     
==========================================
- Hits        31252    27907    -3345     
- Misses         55     2537    +2482     
- Partials        6       18      +12     
Files Changed Coverage Δ
src/chains/definitions/base.ts 100.00% <100.00%> (ø)
src/chains/definitions/baseGoerli.ts 100.00% <100.00%> (ø)

... and 125 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@moldy530 moldy530 left a comment

Choose a reason for hiding this comment

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

LG from alchemy side of things

@jxom jxom merged commit d697293 into wevm:main Sep 10, 2023
5 of 11 checks passed
@github-actions github-actions bot mentioned this pull request Sep 10, 2023
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

3 participants