Bump to ethers v6, wagmi/viem/rainbowkit v2 #125
Merged
Conversation
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
dtbuchholz
commented
Apr 26, 2024
|
|
||
| // All of the chains configured below are supported by Tableland | ||
| // @ts-expect-error this is valid; the OP chains cause a type error | ||
| const chains: readonly [Chain, ...Chain[]] = [ |
There was a problem hiding this comment.
here's the bulk of the changes—it's just updating to the lastest wagmi etc setup and includes all of the newest chains
|
note: i removed node 16 from GH actions since we did that in other packages, too. but, it looks like the workflows prior to making that change are hanging this up. |
joewagner
approved these changes
May 7, 2024
Something weird going on there. test (16) shouldn't be running since you removed it in this PR, and lint is just hanging. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Bumps to new SDK, ethers, and wagmi/viem/rainbowkit v2. Closes ENG-843.
Note: we need to add a
WALLET_CONNECT_PROJECT_IDenv var to this repo due to the newer versions requiring this. I don't have the ability to do that.Details
The only real changes are in
app/components/Providers.tsx. Everything else is eslint fixing files that never got linted.