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

Add missing plural inflections #2116

Conversation

DimitriPapadopoulos
Copy link
Collaborator

I've left out webaservers->webservers, web servers, because I cannot find any occurrence out there (except a webaservers.net domain) and this typo seems highly unlikely to me.

@DimitriPapadopoulos
Copy link
Collaborator Author

Addresses the plural inflections listed in #2079, but there may be more of them.

@vikivivi
Copy link
Contributor

@DimitriPapadopoulos
Copy link
Collaborator Author

DimitriPapadopoulos commented Oct 26, 2021

My wrong, I could just remove weaponaries. Neither the typo weaponaries nor weaponries are found in code out there (except in dictionaries):

I did find a few occurrences of weaponries with Google, although it suggests using weaponry instead.
https://www.google.com/search?q=weaponries
Among them is a scientific paper using weaponries:
Secretome weaponries of Cochliobolus lunatusinteracting with potato leaf at different temperature regimes reveal a CL[xxxx]LHM - motif

@vikivivi
Copy link
Contributor

Searching weaponaries still resulted with some 54 hits. IMO, I think it is fine.
https://github.com/search?q=weaponaries&type=code

@skangas
Copy link
Collaborator

skangas commented Dec 16, 2022

LGTM.

@DimitriPapadopoulos
Copy link
Collaborator Author

Rebased.

@DimitriPapadopoulos
Copy link
Collaborator Author

Rebased.

@DimitriPapadopoulos
Copy link
Collaborator Author

@luzpaz Time permitting, could you have a look at this merge request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review dictionary Changes to the dictionary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants