Skip to content

Spellcheck: fix typos / extend allowlist#50

Merged
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:fix-spellcheck
Jun 13, 2026
Merged

Spellcheck: fix typos / extend allowlist#50
ChrisRackauckas merged 1 commit into
SciML:mainfrom
ChrisRackauckas-Claude:fix-spellcheck

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Contributor

Fixes the failing Spell Check (crate-ci/typos@v1.47.0) CI.

Reproduced locally with typos 1.47.0. Findings and dispositions:

Real prose/comment typos fixed:

  • objct -> object (test/Reactor_tests/CSTR_test.jl comment)
  • pressue -> pressure (src/Separation/Flash.jl description string)
  • pbject -> object (src/Sources/MaterialSource.jl comment)

False positive allowlisted in .typos.toml:

  • Construtor — correct Portuguese spelling of "constructor" in a Portuguese-language source comment (src/Reactors/ReactionManager/KineticReaction.jl: # Construtor para aceitar palavras-chave). Not anglicized so the comment stays valid Portuguese.

typos now exits 0 with zero findings.

Ignore until reviewed by @ChrisRackauckas.

Fix real prose/comment typos flagged by crate-ci/typos (objct -> object,
pressue -> pressure in a description string, pbject -> object). Allowlist
`Construtor` (correct Portuguese spelling of "constructor" in a
Portuguese-language source comment, a false positive).

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ChrisRackauckas ChrisRackauckas marked this pull request as ready for review June 13, 2026 08:58
@ChrisRackauckas ChrisRackauckas merged commit e4c982c into SciML:main Jun 13, 2026
4 of 6 checks passed
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