Skip to content

Fix over 100 typos/spelling mistakes :P #1483

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

Merged
merged 4 commits into from
Nov 23, 2016

Conversation

mnoorenberghe
Copy link
Contributor

Description:
Partially automated identification of words not in the aspell dictionary followed by manual correction. I was a bit bored while waiting for reviews on my code pull requests and noticed quite a few documentation spelling mistakes in the last week since I started using hass so decided to help fix the problem in a larger way than my single-word fixes.

Some of the tools:
find source/_components -name "*.markdown" -exec cat {} \; | aspell list | sort | uniq -c | sort -n # list non-dictionary words
function f { grep -w -R "$1" source/_components/; } # show the typo in context
function r { grep -w -l -R "$1" source/_components/ | xargs -n1 sed -E -i'' "s/\b$1\b/$2/g"; } # replace the typo

@mention-bot
Copy link

@mnoorenberghe, thanks for your PR! By analyzing the history of the files in this pull request, we identified @fabaff, @w1ll1am23 and @Khabi to be potential reviewers.

@Landrash Landrash self-assigned this Nov 23, 2016
Removed extra spaces from emulated_hue
Fix introduced spelling mistake.
@Landrash
Copy link
Contributor

Love that you did this. Fixed a few more things and that one spelling mistake you introduced ;)
Thank you @mnoorenberghe 🍪

@Landrash Landrash merged commit 2b5d3c0 into home-assistant:current Nov 23, 2016
@mnoorenberghe
Copy link
Contributor Author

mnoorenberghe commented Nov 23, 2016

Thanks for the quick reviews and good catch on my typo!

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.

3 participants