Skip to content
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

WIP: Correct all typos #498

Merged
merged 1 commit into from
Jul 20, 2022
Merged

WIP: Correct all typos #498

merged 1 commit into from
Jul 20, 2022

Conversation

shmup
Copy link
Contributor

@shmup shmup commented Jul 20, 2022

Having fun with this one. I'm using this tool: https://github.com/crate-ci/typos

It's a very manual approach. Not letting the tool correct anything.

Generally it's comments being corrected, in much less cases it's a variable, and in one case so far it's a filename.

@shmup shmup force-pushed the correct_typos branch 4 times, most recently from 0dc659c to e4eadb0 Compare July 20, 2022 16:55
@jart jart marked this pull request as ready for review July 20, 2022 18:06
libc/nt/master.sh Outdated Show resolved Hide resolved
libc/nt/ntdll/NtIsUILanguageComitted.s Outdated Show resolved Hide resolved
libc/nt/pedef.internal.h Outdated Show resolved Hide resolved
third_party/tidy/parser.c Outdated Show resolved Hide resolved
third_party/zip/zipcloak.c Outdated Show resolved Hide resolved
@shmup shmup force-pushed the correct_typos branch 2 times, most recently from 365f772 to 7296e11 Compare July 20, 2022 19:52
@jart
Copy link
Owner

jart commented Jul 20, 2022

There's a build error on the changes to the Windows NT constants. This shouldn't matter since it's not part of the ABI. It'll be quicker for me to fix the build myself than to review this change thrice. Thanks for fixing all of my typos! In the future please try to run make -j16 before pushing.

@jart jart merged commit ed205e9 into jart:master Jul 20, 2022
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