Uh oh!
There was an error while loading. Please reload this page.
Add several spelling corrections - #1362
Merged
larsoner merged 1 commit intoDec 6, 2019
Merged
Conversation
luzpaz
approved these changes
Dec 5, 2019
larsoner
reviewed
Dec 5, 2019
Uh oh!
There was an error while loading. Please reload this page.
The corrections are based on the following real world examples: autoconpleting, momemtarily
c72578force-pushed
the
2019-12-05_Add_several_spelling_corrections
branch
from
December 5, 2019 18:47
4c0e06b to
0d728ceComparelarsoner added a commit
that referenced
this pull request
Apr 6, 2020
* Check errors don't exist as valid words in the aspell dictionary * Install aspell on Travis * Add some missing packages * Remove a virtual package * Just install the version of aspell-python we need * Keep flake8 happy * Switch to warnings and count them, so we can see all the aspell errors in one go * Handle different encoding of the word and aspell * Try and fix the encoding conversion * Find out the encoding type * Don't assert on number of warnings * Don't record warnings for now * Warn on all the encoding options * pprint the encoding * More warning work * Use the actual encoding type * Correct the logic * ENH: Multi dict support * FIX: Fixes after merge * FIX: Better error check * FIX: More thorough testing, locations * FIX: Try newer aspell * FIX: Move to new dict * FIX: Move * FIX: Restore removals from #1181 * FIX: One from #1362 * Add rare chack->check, cheque, * Minor tidy of some dictionary check code * Add some more suggestions. * Fix the whitespace * Really fix the whitespace * FIX: Refactor requirement * Log an error when aspell not found and not required * Fix the error logging * Test all variants of present and missing from aspell * Undo some tuple tidying * Fix the true/false values used * Skip some flake8 tests * Fix the test cases * Correct the not in aspell test and fix some test cases * Remove a duplicate test * Use a test word that isn't a typo * Set the ideal aspell detection logic for each dictionary I suspect we'll have to relax this as more obscure words won't be in the aspell dictionary * Be more realistic given the size of the dictionary * Fix a flake8 error * Fix another line length error * FIX: Move * FIX: Make visible, simplify Co-authored-by: Eric Larson <larson.eric.d@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
The corrections are based on the following real world examples:
autoconpleting, momemtarily