Skip to content

Fix wagmi dependency#60

Merged
GiovaniGuizzo merged 1 commit into
mainfrom
fix/dependencies
Oct 29, 2025
Merged

Fix wagmi dependency#60
GiovaniGuizzo merged 1 commit into
mainfrom
fix/dependencies

Conversation

@GiovaniGuizzo

Copy link
Copy Markdown
Collaborator

Description

Adds wagmi dependency to the dev deps so that build won't fail.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation (updates documentation on the project)
  • chore (Updates on dependencies, gitignore, etc)
  • test (For updates on tests)

Copilot AI review requested due to automatic review settings October 29, 2025 19:30
@coderabbitai

coderabbitai Bot commented Oct 29, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

The pull request updates development dependencies in the root package.json by upgrading @types/node from ^20.12.7 to ^24.9.2 and typescript from ^5.1.6 to ^5.9.3. Additionally, packages/evm/package.json adds wagmi (version ^2.18.2) to both peerDependencies and devDependencies, while maintaining viem at the 2.x version range.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify compatibility between upgraded @types/node (^24.9.2) and typescript (^5.9.3) versions
  • Confirm wagmi (^2.18.2) has no breaking changes or conflicting dependencies with existing packages
  • Ensure wagmi is appropriately declared in both peerDependencies and devDependencies for packages/evm/package.json
  • Validate that viem 2.x remains compatible with the newly added wagmi dependency
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/dependencies

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8cdbc3f and b743194.

⛔ Files ignored due to path filters (2)
  • .yarn/install-state.gz is excluded by !**/.yarn/**, !**/*.gz
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
📒 Files selected for processing (2)
  • package.json (2 hunks)
  • packages/evm/package.json (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates key development dependencies to newer versions:

  • Upgrades TypeScript from ^5.1.6 to ^5.9.3
  • Updates @types/node from ^20.12.7 to ^24.9.2
  • Adds wagmi ^2.18.2 as a dev dependency to the packages/evm package

The changes also include a complete yarn.lock update reflecting these dependency version changes and their transitive dependencies.

Reviewed Changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
package.json Updated TypeScript and @types/node versions in root devDependencies
packages/evm/package.json Added wagmi as a devDependency
yarn.lock Updated lockfile with new dependency versions and their transitive dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GiovaniGuizzo GiovaniGuizzo merged commit df48af6 into main Oct 29, 2025
8 of 13 checks passed
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.

3 participants