Skip to content

Conversation

@ChrisRackauckas
Copy link
Member

Summary

  • Fixed multiple spelling errors across test files and documentation
  • Added comprehensive .typos.toml configuration to prevent future false positives

Spelling Corrections Made

  • condtioncondition in test/common_interface/callbacks.jl (2 instances)
  • noticablenoticeable in src/common_interface/algorithms.jl (5 instances)
  • consistant/inconsistantconsistent/inconsistent in test/common_interface/ida.jl (3 instances)
  • refectedreflected in test/common_interface/ida.jl (1 instance)
  • initalinitial in test/common_interface/ida.jl (2 instances)
  • sensivity/sensivitiessensitivity/sensitivities in test/cvodes_dns.jl (5 instances)
  • paramaterparameter in test/cvodes_dns.jl (1 instance)

Configuration Added

  • Added .typos.toml to allow legitimate technical terms like yout (y output) and ypout (y-prime output) which are standard variable names in the Sundials API

Test Plan

  • Verify all changes maintain original functionality
  • Check that legitimate technical terms are preserved
  • Confirm spell checker passes with new configuration

🤖 Generated with Claude Code

- Fix 'condtion' → 'condition' in test/common_interface/callbacks.jl
- Fix 'noticable' → 'noticeable' in src/common_interface/algorithms.jl
- Fix 'consistant/inconsistant' → 'consistent/inconsistent' in test/common_interface/ida.jl
- Fix 'refected' → 'reflected' in test/common_interface/ida.jl
- Fix 'inital' → 'initial' in test/common_interface/ida.jl
- Fix 'sensivity/sensivities' → 'sensitivity/sensitivities' in test/cvodes_dns.jl
- Fix 'paramater' → 'parameter' in test/cvodes_dns.jl
- Add .typos.toml configuration to prevent false positives for legitimate terms like 'yout' (y output)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas merged commit ee8c02c into master Jul 29, 2025
4 of 5 checks passed
@ChrisRackauckas ChrisRackauckas deleted the fix-typos-sundials branch July 29, 2025 17:27
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.

2 participants