Fix typos 2 #10162
Merged
Fix typos 2 #10162
+969
−297
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MarekM25
reviewed
Jan 9, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request fixes numerous typos across the Nethermind codebase and adds a cspell-based spelling check to the CI pipeline. The PR includes:
- Correction of method names (e.g.,
DitributeFunds→DistributeFunds,NoitifyAboutTx→NotifyAboutTx) - Variable name improvements for clarity (e.g.,
addrs→addresses,pvkey→pvKey) - Comment and documentation typo fixes throughout the codebase
- Migration of cspell configuration from
src/Nethermind/cspell.jsonto root-levelcspell.jsonwith expanded word list - Addition of cspell GitHub Actions workflow step to enforce spelling checks
Reviewed changes
Copilot reviewed 114 out of 114 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/code-formatting.yml |
Adds cspell action to CI workflow |
cspell.json |
New root-level spelling configuration with comprehensive word list |
src/Nethermind/cspell.json |
Removed (moved to root) |
tools/SendBlobs/*.cs |
Fixed method name typos (DitributeFunds → DistributeFunds) |
tools/Kute/README.md |
Clarified pronunciation notation |
src/Nethermind/Nethermind.Xdc*/*.cs |
Multiple typo fixes in variable names and comments |
src/Nethermind/Nethermind.TxPool/*.cs |
Fixed method name NoitifyAboutTx → NotifyAboutTx across interface and implementations |
src/Nethermind/Nethermind.Network/*.cs |
Variable name improvements and typo fixes |
src/Nethermind/Nethermind.JsonRpc/*.cs |
Documentation and comment typo fixes |
src/Nethermind/Nethermind.Evm/*.cs |
Comment corrections and typo fixes |
src/Nethermind/Nethermind.Core/*.cs |
Comment and documentation improvements |
| Other test and implementation files | Consistent typo corrections throughout |
Co-authored-by: Copilot <[email protected]>
LukaszRozmej
approved these changes
Jan 9, 2026
flcl42
requested review from
a team,
Demuirgos,
Marchhill,
ak88,
asdacap,
benaadams,
cicr99,
damian-orzechowski,
deffrian,
emlautarom1,
marcindsobczak,
rubo and
smartprogrammer93
as code owners
flcl42
force-pushed
the
fix-typos-2
branch
3 times, most recently
from
3b805dc to
cf40cdd
Compare
benaadams
approved these changes
Jan 9, 2026
MarekM25
approved these changes
Jan 9, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
If yes, did you write tests?