-
-
Notifications
You must be signed in to change notification settings - Fork 197
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: Issue #5286: Add some words that start with "un" to the US dictionary #3129
Conversation
Fixes streetsidesoftware/cspell#5286 CSpell has been nagging about these words found in the Backdrop CMS codebase.
Thank you for taking a look at this. Some of the terms are more programming related than part of general English. For example, what does The other drawback to adding them to the US English dictionary is that they are only available to those who use If a term is primarily used when writing code, please add it to dictionaries/software-terms/src/software-terms.txt. If you feel it is a universal term (all flavors of English), then consider adding it to dictionaries/en_shared/src/shared-additional-words.txt |
Will you be able to adjust the PR as suggested? |
@Jason3S sorry for being unresponsive here, and yes, I plan to loop back to this and adjust as per your suggestions. We have been busy in the Backdrop CMS project, with the feature freeze being on May 1st, and now the upcoming release in May 15. I'm hoping to slate some time for this on the weekend of May 18-19. I would appreciate it if you kept this open if not too much trouble please. |
No a problem. I'll keep the PR open. I appreciate the help. |
@klonos, |
@@ -54,12 +55,33 @@ rebrand | |||
rebranding | |||
teleconverter | |||
tricep | |||
unaggregated | |||
uncustomized |
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.
uncontained
is missing, found it today!
Closes #3129 fixes: streetsidesoftware/cspell#5286
Fixes streetsidesoftware/cspell#5286
Add/Fix Dictionary
Dictionary: US
Description
CSpell has been nagging about these words found in the Backdrop CMS codebase, but they are not specific to that.
It also adds the word
instantiatable
, since it is an actual word (see https://en.wiktionary.org/wiki/instantiatable), as well as itsuninstantiatable
counterpart.References
backdrop/backdrop-issues#6302
Checklist
fix:
- for minor changes like adding words or fixing spelling issues.feat:
- for a significant change like adding a whole new set of words to a dictionary.feat!:
- for breaking changes, like file format or licensing changes.chore:
- for changes that do not impact the content of dictionaries.