Skip to content

docs: fix typos in jsdoc comments and one error message#5136

Open
0xAxiom wants to merge 1 commit into
ethers-io:mainfrom
0xAxiom:fix/typos-jsdoc-comments
Open

docs: fix typos in jsdoc comments and one error message#5136
0xAxiom wants to merge 1 commit into
ethers-io:mainfrom
0xAxiom:fix/typos-jsdoc-comments

Conversation

@0xAxiom

@0xAxiom 0xAxiom commented May 6, 2026

Copy link
Copy Markdown

Pure typo fixes in src.ts/ (no behavior change — build artifacts in lib.esm/ and lib.commonjs/ are regenerated by the build, so they were left alone here).

Changes

File Fix
abi/interface.ts "the the Contract with the sepcified""the Contract with the specified"
abi/fragments.ts "wich""which", "frgament""fragment" (jsdoc + one assertArgument error message)
contract/wrappers.ts "occured""occurred" (×3 in ContractUnknownEventPayload jsdoc)
providers/abstract-signer.ts "teh""the"
providers/provider-fallback.ts "occured""occurred"
providers/default-provider.ts "gaurantee""guarantee", "qorum""quorum"

11 typos across 6 files; +11/-11 lines.

The only non-comment change is the assertArgument(false, "unsupported frgament object", ...) message in fragments.ts — flagging in case the exact error string is asserted on anywhere downstream (a quick search in this repo shows it isn't), happy to drop that one if preferred.

Commit is GPG-signed.

Pure typo fixes in src.ts (build artifacts in lib.esm/lib.commonjs will
be regenerated):

- abi/interface.ts: "the the Contract with the sepcified" -> "the Contract with the specified"
- abi/fragments.ts: "wich" -> "which", "frgament" -> "fragment" (jsdoc + assertArgument message)
- contract/wrappers.ts: "occured" -> "occurred" (3x in EventPayload jsdoc)
- providers/abstract-signer.ts: "teh" -> "the"
- providers/provider-fallback.ts: "occured" -> "occurred"
- providers/default-provider.ts: "gaurantee" -> "guarantee", "qorum" -> "quorum"

No code/behavior changes other than the assertArgument error message
text (also a typo fix).
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.

1 participant