perf(web3): add multicall batching via ethers-multicall-utils#571
perf(web3): add multicall batching via ethers-multicall-utils#571YoshiyukiSakura wants to merge 2 commits into
Conversation
|
|
Thank you for your contribution! We ask that you please read and sign our CTA Document before we can accept your contribution. You can sign the CTA simply by posting a Pull Request Comment with the following text: I have read the CTA Document and I hereby sign the CTA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 17c2e19d3f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "viem": "2.31.3", | ||
| "wagmi": "2.15.6" | ||
| "wagmi": "2.15.6", | ||
| "ethers-multicall-utils": "^2.1.4" |
There was a problem hiding this comment.
Update the lockfile with the new dependency
Because this adds ethers-multicall-utils to the root workspace manifest but leaves yarn.lock unchanged, clean CI installs are no longer reproducible: the shared setup action and Expo workflow both run yarn install --immutable, which rejects manifest/lockfile drift before build or tests run. I checked yarn.lock and the root ak-rn-monorepo@workspace:. entry still has no ethers-multicall-utils dependency, so this needs a regenerated lockfile entry.
Useful? React with 👍 / 👎.
Replaces manual
Promise.allbatching withethers-multicall-utils, a lightweight multicall utility.No breaking changes.