Navigation Menu

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

Spellchecking #28232

Merged
merged 1 commit into from Feb 18, 2019
Merged

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Fixes some typos.

data/json/mapgen/lot_empty_residential.json Outdated Show resolved Hide resolved
data/mods/Salvaged_Robots/items.json Outdated Show resolved Hide resolved
data/mods/blazemod/blaze_blob.json Outdated Show resolved Hide resolved
data/json/items/gunmod/grip.json Outdated Show resolved Hide resolved
@Wuzzy2
Copy link
Contributor

Wuzzy2 commented Feb 17, 2019

May I ask what script/command you used for this? :-)

Maybe translators could use the same trick to spellcheck the translations.

@kevingranade
Copy link
Member Author

kevingranade commented Feb 17, 2019

@Wuzzy2 as per your suggestion, I used aspell, specifically: aspell -c lang/po/cataclysm-dda.pot. I had one terminal open running that, and a different terminal with the dda source open where I would search for misspelled words, which was usually pretty straightforward since misspelled words tend to be unique. You can probably do the same in .po (or mo? I can't keep them straight) files with the appropriate language dictionary and correct found issues in the same file.

It was not fun, we use massive amounts of jargon, acronyms and "novel" conjugations of words, so the false negative rate for things aspell complained about that weren't issues was massive. If I had to guess I'd say it was something like 500 false negatives per found misspelling.

@Rivet-the-Zombie Rivet-the-Zombie merged commit 3c1a288 into CleverRaven:0.D-branch Feb 18, 2019
@kevingranade kevingranade deleted the spellchicking branch March 26, 2019 23:44
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.

None yet

5 participants