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 typo: indezes => indeces #11953

Merged
merged 1 commit into from
Jun 20, 2022
Merged

fix typo: indezes => indeces #11953

merged 1 commit into from
Jun 20, 2022

Conversation

AbdelrahmanHafez
Copy link
Collaborator

No description provided.

indezes is probably German, we could also use indexes but indeces is more comoon
@AbdelrahmanHafez AbdelrahmanHafez requested review from vkarpov15 and Uzlopak and removed request for vkarpov15 June 19, 2022 21:40
Copy link
Collaborator

@Uzlopak Uzlopak left a 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.

@AbdelrahmanHafez AbdelrahmanHafez added this to the 6.4.1 milestone Jun 20, 2022
@AbdelrahmanHafez AbdelrahmanHafez merged commit 52e4d52 into master Jun 20, 2022
@AbdelrahmanHafez AbdelrahmanHafez added the typescript Types or Types-test related issue / Pull Request label Jun 20, 2022
@Uzlopak Uzlopak deleted the indeces-typo branch June 20, 2022 09:16
@@ -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" />
Copy link
Collaborator

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.

Copy link
Collaborator

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?

Copy link
Collaborator Author

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.

Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typescript Types or Types-test related issue / Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants