Skip to content

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

Merged
drgrice1 merged 1 commit into
openwebwork:WeBWorK-2.21from
Alex-Jordan:typos
Jul 7, 2026
Merged

typos identified by github.com/crate-ci/typos#3031
drgrice1 merged 1 commit into
openwebwork:WeBWorK-2.21from
Alex-Jordan:typos

Conversation

@Alex-Jordan

Copy link
Copy Markdown
Contributor

These are typos identified by github.com/crate-ci/typos, except not typos in lib. Except a few that are in lib that I either already had on my typos branch, or that are connected to a typo in templates and can only be fixed at the same time.

@Alex-Jordan

Copy link
Copy Markdown
Contributor Author

github.com/crate-ci/typos identifies 1185 typos in lib. I'm sure some of them are false positives, but I would not be surprised if around 1000 of them are legit. I'll save that for some other time. Or anyone else, feel free to try this tool. I'll be offline for a few weeks now.

Comment thread doc/Artistic Outdated
@somiaj

somiaj commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

I looked over the typos. Most are just comments/output text and look fine. A few did change some typos in variable names. In this case they appear to have found all cases, because it looks like the variable names were consistently spelled wrong in the code. Though I didn't do any testing and some should be done to ensure that code changes didn't break anything (initial look is most likely not).

@pstaabp

pstaabp commented Jul 7, 2026

Copy link
Copy Markdown
Member

Looking over this, it seems that there are only two files with code ramifications, PGProblemEditor.pm and the template save_as_form.html and since the typos were consistent in the previous versions and fixed in both places, this should be good.

Should we also then revert the typo in doc/Artistic ?

@Alex-Jordan

Copy link
Copy Markdown
Contributor Author

@pstaabp I just now reverted that license typo correction.

@drgrice1 drgrice1 merged commit 4d58ba2 into openwebwork:WeBWorK-2.21 Jul 7, 2026
2 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.

4 participants