-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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 typo: indezes => indeces #11953
fix typo: indezes => indeces #11953
Conversation
indezes is probably German, we could also use indexes but indeces is more comoon
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.
I am ok with this.
@@ -8,7 +8,7 @@ | |||
/// <reference path="./expressions.d.ts" /> | |||
/// <reference path="./helpers.d.ts" /> | |||
/// <reference path="./middlewares.d.ts" /> | |||
/// <reference path="./indizes.d.ts" /> | |||
/// <reference path="./indeces.d.ts" /> |
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.
Minor nitpick: this is still a typo, the correct spelling is "indices". However, we typically use "indexes" as the plural form of "index". Similar reasoning to the "schemas" vs "schemata" debate: "indexes" is acceptable according to Oxford English Dictionary, and should be easier for non-native English speakers to understand because it doesn't use a special pluralization rule.
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.
Should @AbdelrahmanHafez change it to indexes?
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.
@vkarpov15
Good catch 🤦
I considered both options and considered that indexes is easier for non-native English speakers, but a quick Google search says that indices
is actually more common than indexes
. But yeah, I can see that even though I knew indices
, I don't know how to spell it properly.
We can go for indexes
which is a little bit less acceptable, but expectable.
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.
Another point in favor of "indexes" is that the MongoDB documentation calls them "indexes" rather than "indices": https://www.mongodb.com/docs/manual/indexes/ . I think we should err on the side of consistency with the MongoDB docs on this front
fix(types): fix typo for `indexes` re #11953
No description provided.