[pt] Check English words in speller #10529
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@susanaboatto I've sent you on Slack a sample diff showing the performance of the speller after tagging English words with
_english_ignore_
. If you need a different test set, do let me know.@jaumeortola this logic seems to work for me. The one annoying thing about it is that, since we need the English tagger for the filter to work, I cannot add Java tests without making English a dependency. Any ideas? I'd really like to add some proper tests here.