wevm / abitype

Strict TypeScript types for Ethereum ABIs
https://abitype.dev
MIT License
474 stars 39 forks source link

fix: typos #212

Closed shuoer86 closed 9 months ago

shuoer86 commented 9 months ago

PR-Codex overview

This PR focuses on fixing typos in the abitype package.

Detailed summary:

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

changeset-bot[bot] commented 9 months ago

🦋 Changeset detected

Latest commit: bbe123b9c3459119d57f17841149f90f24f2a7c9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------- | ----- | | abitype | Patch |

Not sure what this means? Click here to learn what changesets are.

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

vercel[bot] commented 9 months ago

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

Name Status Preview Comments Updated (UTC)
abitype ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 28, 2023 2:52pm
shuoer86 commented 9 months ago

@tmm Thanks for your review. I have added changeset.

Raiden1411 commented 9 months ago

Thanks for the PR! Would you mind also updating the typos here: https://github.com/wevm/abitype/blob/00dd694139d995a98b8994ed180e60bcbc5e2947/packages/abitype/src/human-readable/errors/abiItem.test.ts#L11 https://github.com/wevm/abitype/blob/00dd694139d995a98b8994ed180e60bcbc5e2947/packages/abitype/src/human-readable/errors/abiParameter.test.ts#L15 https://github.com/wevm/abitype/blob/00dd694139d995a98b8994ed180e60bcbc5e2947/packages/abitype/src/human-readable/errors/abiParameter.test.ts#L27 https://github.com/wevm/abitype/blob/00dd694139d995a98b8994ed180e60bcbc5e2947/packages/abitype/src/human-readable/errors/abiParameter.test.ts#L40 https://github.com/wevm/abitype/blob/00dd694139d995a98b8994ed180e60bcbc5e2947/packages/abitype/src/human-readable/errors/abiParameter.test.ts#L118

shuoer86 commented 9 months ago

@Raiden1411 updated, thanks for your review.