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 words to hyphenated words list #782

Merged
merged 1 commit into from
Jan 6, 2025
Merged

Conversation

vr8hub
Copy link
Contributor

@vr8hub vr8hub commented Jan 6, 2025

A few more words run across either in reviews or in my current production, all in M-W.

While adding mantelshelf, I noticed we have both mantelpiece(s) and mantlepiece(s). M-W says (scroll down to the Did You Know?) that mantel and mantle aren't really synonyms, although the latter is sometimes used, "especially in American English," for the former. Ngrams reflects that; mantlepiece is barely above zero, mantelpiece is a couple of order of magnitudes more common. (In English fiction; it's even more pronounced in English in general.)

So, I wonder if we should change mantle-?piece to mantelpiece in spelling.py, and remove mantlepiece(s) from words? There are 45 occurrences of mantlepiece in the corpus that would need to be changed.
Ditto for mantle-?shelf; we have four of those in the corpus, vs. 117 of mantelshelf.

cotehardie(s)
mantelshelf
mantelshelves
tumbledown

cotehardie(s)
mantelshelf
mantelshelves
tumbledown
@acabal
Copy link
Member

acabal commented Jan 6, 2025

Sure, I think we can do that if you can open a PR for the toolset.

@acabal acabal merged commit b3bc00d into standardebooks:master Jan 6, 2025
1 check passed
@vr8hub vr8hub deleted the words branch January 7, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants