Skip to content

[ci skip] Fix spelling errors in NonlinearSolve.jl #653

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 29, 2025

Conversation

ChrisRackauckas
Copy link
Member

Summary

This PR fixes spelling errors found by the typos spell checker.

Changes Made

  • LSO -> ALSO in ext/NonlinearSolveLeastSquaresOptimExt.jl
  • theorectically -> theoretically in lib/NonlinearSolveBase/src/descent/geodesic_acceleration.jl
  • copyies -> copies in lib/NonlinearSolveBase/src/tracing.jl

Notes

  • included to avoid unnecessary CI runs for spelling fixes
  • All changes are simple typo corrections that don't affect functionality
  • Changes were made using automated spell checking with manual review

claude added 2 commits July 29, 2025 11:26
…eastSquaresOptimExt.jl\n- theorectically -> theoretically in lib/NonlinearSolveBase/src/descent/geodesic_acceleration.jl\n- copyies -> copies in lib/NonlinearSolveBase/src/tracing.jl
- Add LSO to .typos.toml as allowed term
- LSO is a legitimate abbreviation used in optimization contexts
- Revert incorrect ALSO changes back to LSO in extension file
- LSO should not be treated as a typo
@ChrisRackauckas ChrisRackauckas merged commit f4b7b05 into master Jul 29, 2025
72 of 92 checks passed
@ChrisRackauckas ChrisRackauckas deleted the fix-spelling-issues branch July 29, 2025 15:32
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