-
Notifications
You must be signed in to change notification settings - Fork 475
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
These typos do not belong to code, do they? #3470
Conversation
34fadb5
to
8f5c0a6
Compare
8f5c0a6
to
e06e729
Compare
I have left a handful changes. |
e06e729
to
1558b77
Compare
While the suggested fixes are words often used in code, the typos themselves do not seem to be valid words - even in code.
1558b77
to
df4c1d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@peternewman could you review this?
some of these are misspellings of commonly used coding terminology. |
Yes, hence why they've been moved to the main dictionary @luzpaz . The code dictionary is only for where the typo is valid in code but a typo in normal usage. So technically it's the !code dictionary! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks @DimitriPapadopoulos !
While the suggested fixes are words often used in code, the typos themselves do not seem to be valid words - even in code.
Fixes #3430 (comment).