-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 a spellchecker #357
Comments
One thing I want to really stress here is that I want to avoid an english-centric solution at all cost. I think it would be a shame to lock ourselves in with a solution for "the majority". For the rest, I don't know much about spellchecking tools, so I'm open to everything. I also think this is a very desirable feature. Things to think about:
|
I want to work on this. Any guidelines? |
Now that @prertik, are you still interested in working on this? Rough OutlineMy thoughts are you'd use something like the Thanks to #541 users will be able to temporarily change the mode via an environment variable. So you could invoke The project-specific dictionary should probably live next to You can use pulldown-cmark to scan through and only check the text sections, if so desired. I would recommend having a look at the spellcheck.sh used for the second edition of the book. This gives you a rough outline of what's being done already. To make sure this isn't english-centric, you'd want to provide a Configuration might look something like this: [output.ispell]
language = "en"
mode = "check"
project-dictionary = "words.txt" |
I'd love to but I'm sorry @Michael-F-Bryan I currently have very tight schedule. Thank you for the awesome guideline. |
This is quite a complex feature to add, and it's frankly a lot easier to just install a spellchecker in your IDE or utilize an existing markdown spellcheck tool. Thoughts on closing? |
It would be nice to have a spellchecker (in cookbook we constantly fight with typos).
The rust book uses a shell script running aspell.
We might consider using https://crates.io/crates/ispell as part of
mdbook test
(and possibly as a warning inmdbook build
)The text was updated successfully, but these errors were encountered: