-
-
Notifications
You must be signed in to change notification settings - Fork 559
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
refactor: remove defineChain export #424
Conversation
🦋 Changeset detectedLatest commit: e0b22e8 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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
0dcc4a4
to
e0b22e8
Compare
Size Change: -66 B (0%) Total Size: 252 kB
ℹ️ View Unchanged
|
Codecov Report
@@ Coverage Diff @@
## main #424 +/- ##
=======================================
Coverage 99.93% 99.93%
=======================================
Files 267 267
Lines 22655 22654 -1
Branches 1925 1924 -1
=======================================
- Hits 22640 22639 -1
Misses 15 15
|
This is technically a breaking change. Don't think anyone would be using it yet though. Maybe we can move the export to the root entrypoint? |
Yeah. I think it's better to do this now than later. Also, it's a very small breaking change considering that all you'd have to do to update is to change the import. I'll add it to \Wait... It already is in the root entrypoint! |
Yeah, so imho this would be ready then? Unless I'm missing sth. :-) |
PR-Codex overview
This PR removes the
defineChain
export fromviem/chains
and removesdefineChain
import from multiple files.Detailed summary
defineChain
export fromviem/chains
.defineChain
import fromsrc/chains.ts
.defineChain
import fromsrc/actions/wallet/sendTransaction.test.ts
.