wevm / wagmi

React Hooks for Ethereum
https://wagmi.sh
MIT License
5.79k stars 968 forks source link

fix: correct project typos #4102

Closed ClumsyVlad closed 6 days ago

ClumsyVlad commented 1 week ago

After analyzing the project code, typos in explanation comments and documentation files were found and corrected.

changeset-bot[bot] commented 1 week ago

⚠️ No Changeset found

Latest commit: 041a6fc37377951bd52d8027513196731d58d329

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wagmi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 1, 2024 10:38pm
ClumsyVlad commented 1 week ago

@tmm could you explain, please, why there should be the intentioan typo in balanceOf function name? And also, why would you like to keep intentional typo in comment in watchChains.ts? 🤔

tmm commented 1 week ago

why there should be the intentioan typo in balanceOf function name?

This typo is intentional to showcase Wagmi's type-safety.

image

why would you like to keep intentional typo in comment in watchChains.ts?

This is a joke we'd like to keep around.

ClumsyVlad commented 6 days ago

This typo is intentional to showcase Wagmi's type-safety.

Reverted 🫡

This is a joke we'd like to keep around.

Reverted 🫡