Skip to content

typos identified by github.com/crate-ci/typos#1448

Open
Alex-Jordan wants to merge 5 commits into
openwebwork:PG-2.21from
Alex-Jordan:typos
Open

typos identified by github.com/crate-ci/typos#1448
Alex-Jordan wants to merge 5 commits into
openwebwork:PG-2.21from
Alex-Jordan:typos

Conversation

@Alex-Jordan

Copy link
Copy Markdown
Contributor

This is a massive collection of typo corrections after I used github.com/crate-ci/typos. Note that it had some false positives, and I examined each one of these to avoid the false positives.

Most of these are in code comments and POD.

A small number of these actually correct a meaningful typo in code.

An even smaller number of these correct some typo in a variable or method name. In those cases, I checked that nothing else was relying on the bad variable/method name.

@drgrice1 drgrice1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good, but the code changes need to be checked carefully.

Comment thread macros/contexts/contextPolynomialFactors.pl
Comment thread macros/math/SimpleGraph.pl Outdated
@Alex-Jordan

Copy link
Copy Markdown
Contributor Author

I'm separating this into commits that hopefully make it easier to review. First one is meant to only be POD typos.

@Alex-Jordan

Copy link
Copy Markdown
Contributor Author

No promises that I correctly sorted these, but I tried. The POD ones were hardest to categorize with accuracy when just manually looking at a small diff.

@pstaabp

pstaabp commented Jul 7, 2026

Copy link
Copy Markdown
Member

Everything looks good except for the two macros:

  • lib/Parser/Legacy/LimitedComplex.pm
  • macros/contexts/contextPolynomialFactors.pl

which has a typo in the name. It may be easier to check them separately. How about pull those out and put them in separately?

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.

3 participants