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

Fix some typo in source. #82778

Closed
wants to merge 1 commit into from
Closed

Fix some typo in source. #82778

wants to merge 1 commit into from

Conversation

emacser0
Copy link
Contributor

@emacser0 emacser0 commented Oct 4, 2023

"wihout" -> "without"
"mutli" -> "multi"
"maxium" -> "maximum"
"childs" -> "children"

@emacser0 emacser0 requested review from a team as code owners October 4, 2023 10:27
@emacser0 emacser0 closed this Oct 4, 2023
@akien-mga
Copy link
Member

Hi :) Not sure why the PR was closed, but some of those changes make sense.

We shouldn't modify thirdparty code however, so these would need to be left as is.

We usually fix typos using https://github.com/codespell-project/codespell, so if it missed those, it would be worth making a PR upstream to add them to their dictionary.

Also note that your commit seems not to be linked to your GitHub account. See: Why are my commits linked to the wrong user? for more info.

@AThousandShips
Copy link
Member

Superseded by:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants