-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update to use stable Viem / Wagmi versions #63
Merged
Merged
Changes from 4 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
5196b06
Update tests
zencephalon bd5f860
Fix type errors
zencephalon 4c86edc
Bump to first major release
zencephalon 6ad627d
Lint use of boolean coercion
zencephalon 3452039
Update peer dependency specification
zencephalon f9b8a96
Rewrite useWriteFinalizeWithdrawalTransaction.ts to use Viem config
zencephalon 106e032
Refactor useWriteDepositETH
zencephalon 117c873
Check point, huge refactor
zencephalon 218da87
Add validateChains util
zencephalon 56e8d51
Fix useWriteDepositERC20
zencephalon d09d79d
Remove goerli chains
zencephalon e2b9f04
Update chain configs
zencephalon 01956fe
Use validation in useWriteFinalizeWithdrawalTransaction
zencephalon 4e377ee
Update chain config example
zencephalon db02306
Update error messages
zencephalon efc948c
Try allowing Anvil more time to startup
zencephalon c416e38
Start in parallel, longer deadline
zencephalon 7374a11
Fix vitest failed to terminate worker
zencephalon 87fbf7a
Hopefully fix flaky tests
zencephalon b83eeed
Return the promises to await
zencephalon 69f0c58
Use viem 2.7.11 for l1StandardBridge support, update docs to reflect
zencephalon 1e1c5ae
Delete unnecessary OpConfig code
zencephalon 6d567ef
Update example site
zencephalon cabb074
Refactor example app to use Sepolia, add useSelectedNetwork hook
zencephalon d1e0a8d
Remove lint ala Lukas
zencephalon 1590bec
Add changeset
zencephalon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think these be
2.x
? that's what we have in op-viem and how wagmi has their peer dependencies set upThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can change it to that, my understanding of the
^
operator is that^2.7.9
and^2.x
have the same effect as they both allow any minor and patch version within 2