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

Remove duplicated words #177

Merged
merged 1 commit into from
Jul 19, 2016
Merged

Remove duplicated words #177

merged 1 commit into from
Jul 19, 2016

Conversation

jwilk
Copy link
Contributor

@jwilk jwilk commented Jun 2, 2016

They were found using mwic.

@joeyespo joeyespo merged commit 91e0ee1 into joeyespo:master Jul 19, 2016
@joeyespo
Copy link
Owner

Thanks, @jwilk!

Curious. Did you manually use mwic on this project, or was this an automated PR?

@joeyespo
Copy link
Owner

This was included in the recent v4.3.0 release. Thanks again, @jwilk!

@jwilk
Copy link
Contributor Author

jwilk commented Jul 19, 2016

The former.

mwic only presents the user probable misspelling (with lots of false positives).
It doesn't know how to fix them, and it doesn't even tell you where they are.
Not very useful for automatic PRs. 😏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants