Add codespell support with configuration and fixes - #23
Merged
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a .codespellrc configuration file and fixes various typos in comments and documentation across multiple files, including corrections for 'doesn't', 'identified', 'internally', 'the', 'labels', 'matching', 'preselect', and 'assumes'. There are no review comments to address, and I have no further feedback to provide.
Co-Authored-By: Claude Code 2.1.157 / Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Code 2.1.157 / Claude Opus 4.7 <noreply@anthropic.com>
Fixed ambiguous typos: - interally -> internally (SharpEyes/Models/Calibrator.cs, SharpEyes/Models/RBF2D.cs, Eyetracking/Calibrator.cs, Eyetracking/RBF2D.cs) Comment "they're interally contained" describes encapsulation, i.e. internal use. - matchine -> matching (Eyetracking/Templates.cs) XML doc "<returns>best matchine digit</returns>" refers to the best-matching digit. Co-Authored-By: Claude Code 2.1.157 / Claude Opus 4.7 <noreply@anthropic.com>
Fixed typos:
- doesen't -> doesn't (SharpEyes/SharpEyes.csproj, CrashReporter/CrashReporter.csproj)
- Pre-select -> Preselect (SharpEyes/ViewModels/PythonSettingsViewModel.cs)
- Indentified -> Identified (SharpEyes/Models/CalibrationParameters.cs, Eyetracking/CalibrationParameters.cs)
- tge -> the (Eyetracking/MainWindow.CalibrationTab.cs)
- lables -> labels (Eyetracking/MainWindow.xaml.cs)
- assums -> assumes (docs/workflow/stimulus and gaze.md)
Co-Authored-By: Claude Code 2.1.157 / Claude Opus 4.7 <noreply@anthropic.com>
=== Do not change lines below ===
{
"chain": [],
"cmd": "codespell -w",
"exit": 0,
"extra_inputs": [],
"inputs": [],
"outputs": [],
"pwd": "."
}
^^^ Do not change lines above ^^^
Fixed contextually unambiguous typos introduced in new origin/main content: - frams -> frames (README.md: recentered frames) - fiter -> filter (README.md: filter responses) - Becuase -> Because (README.md) - buil -> build (README.md: build from source) - exectuable -> executable (docs/building.md) Co-Authored-By: Claude Code 2.1.157 / Claude Opus 4.7 <noreply@anthropic.com>
mvdoc
force-pushed
the
worktree-introduce-codespell
branch
from
May 29, 2026 22:34
476c52e to
67741d3
Compare
candytaco
self-requested a review
May 29, 2026 23:31
candytaco
approved these changes
May 29, 2026
# Conflicts: # Eyetracking/CalibrationParameters.cs # Eyetracking/Calibrator.cs # Eyetracking/MainWindow.CalibrationTab.cs # Eyetracking/MainWindow.xaml.cs # Eyetracking/RBF2D.cs # Eyetracking/Templates.cs
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Add codespell configuration and fix existing typos.
More about codespell: https://github.com/codespell-project/codespell
CI workflow has
permissionsset toreadonly so it's safe.Changes
Configuration & Infrastructure
.codespellrcwith skip patterns for binaries (*.pdf,*.svg,*.png,*.ico,*.db), CSS, minified files, lock files, build outputs (bin,obj,packages), and*.DotSettings..github/workflows/codespell.ymlto run codespell on push and PRs targetingmain.Typo Fixes
Ambiguous typos fixed manually (5 fixes with context review):
interally→internallyin 4 files (SharpEyes/Models/Calibrator.cs,SharpEyes/Models/RBF2D.cs,Eyetracking/Calibrator.cs,Eyetracking/RBF2D.cs). Comment "they're interally contained" describes encapsulation of NumSharp structures, so "internally" not "integrally".matchine→matchinginEyetracking/Templates.cs. XML doc<returns>best matchine digit</returns>refers to the best-matching digit, not "best machine digit".Non-ambiguous typos fixed automatically via
datalad run 'codespell -w'(8 fixes across 8 files):doesen't→doesn'tinSharpEyes/SharpEyes.csprojandCrashReporter/CrashReporter.csprojIndentified→Identifiedin twoCalibrationParameters.csfilestge→theinEyetracking/MainWindow.CalibrationTab.cslables→labelsinEyetracking/MainWindow.xaml.csPre-select→PreselectinSharpEyes/ViewModels/PythonSettingsViewModel.csassums→assumesindocs/workflow/stimulus and gaze.mdAll edits are in comments, XML docstrings, or prose — no code identifiers, regex patterns, string literals, or behavior affected.
Testing
Codespell passes with zero errors after all fixes.
Generated with Claude Code.