Skip to content

More conversions from enGB to enUS (after #1485) - #1626

Open
sebweb3r wants to merge 9 commits into
codespell-project:mainfrom
sebweb3r:enUS_aspell
Open

More conversions from enGB to enUS (after #1485)#1626
sebweb3r wants to merge 9 commits into
codespell-project:mainfrom
sebweb3r:enUS_aspell

Conversation

@sebweb3r

Copy link
Copy Markdown
Contributor

The words are in dictionary, but #1485 is needed.

@sebweb3r

Copy link
Copy Markdown
ContributorAuthor

And #1625

@peternewmanpeternewman added the dictionary Changes to the dictionary label Aug 12, 2020
@sebweb3rsebweb3r mentioned this pull request Aug 30, 2020
customised->customized
customising->customizing
daemonisation->daemonization
daemonisation->demonization

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was the left-side supposed to be demonisation ?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This conversion is already moved to #1653. I just messed up.

All corrections here wait for a custom aspell dictionary to pass the tests.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was more referring to the fact that you have two lines with daemonisation on the left side.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. You were right, it was supposed to be demonisation. but I moved the words, that are in aspell to #1653.
I deleted the duplicate entry now. :-)

Comment threadcodespell_lib/data/dictionary_en-GB_to_en-US.txt
Comment threadcodespell_lib/data/dictionary_en-GB_to_en-US.txt
Comment threadcodespell_lib/data/dictionary_en-GB_to_en-US.txt
@lurch

Copy link
Copy Markdown
Contributor

I guess even after the addition of #1485 , words like "daemonisation" will still cause problems? http://app.aspell.net/lookup?dict=en_GB-large&words=daemonisation

@lurch

Copy link
Copy Markdown
Contributor

Sorry for dumping so many comments on your PR, let me know if you'd like me to add them to #1666 instead!

@sebweb3r

Copy link
Copy Markdown
ContributorAuthor

No problem. I added them to a new MR myself, because I have read this to late.

Yes, no, maybe. I'm not sure, which custom dictionary Peter wants to use. But it for sure, that it has to wait atleast until then.

@lurch

lurch commented Sep 2, 2020

Copy link
Copy Markdown
Contributor

@sebweb3r Can you please add these words to this PR?

customiser->customizer
digitiser->digitizer
reinitialisation->reinitialization
reinitialises->reinitializes
reinitialising->reinitializing
virtualise->virtualize
virtualised->virtualized
virtualises->virtualizes
virtualising->virtualizing

I had to strip them out of #1666 and it probably makes sense to bundle them into this PR, rather than having another "after #1485" PR 😉

Comment threadcodespell_lib/data/dictionary_en-GB_to_en-US.txt
@bl-uebl-ue self-assigned this Jun 12, 2021
@bl-ue

Copy link
Copy Markdown
Contributor

I'll fix the conflicts.

@bl-ue

Copy link
Copy Markdown
Contributor

Conflicts fixed 👍🏻

@bl-ue

Copy link
Copy Markdown
Contributor

Ping @lurch: LGTY?

@lurch

Copy link
Copy Markdown
Contributor

Looks okay to me, apart from the failing CI checks 😉

sort: codespell_lib/data/dictionary_en-GB_to_en-US.txt:21: disorder: authorise->authorize
make: *** [Makefile:20: check-dictionaries] Error 1
Dictionary codespell_lib/data/dictionary_en-GB_to_en-US.txt not sorted. Sort with 'make sort-dictionaries'
Error: Process completed with exit code 2.

@bl-ue

Copy link
Copy Markdown
Contributor

Oh darn, I always need to do that after I fix the conflicts 😆

@bl-ue
bl-ue enabled auto-merge (squash) June 13, 2021 12:15
@bl-ue

Copy link
Copy Markdown
Contributor

Ouch, there are a lot of errors 🤔

E error 'authoriser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'customiser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'daemonisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'daemonise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'daemonised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'daemonises' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'deinitialise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'devirtualisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'devirtualise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'devirtualised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'digitiser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'initialisations' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'initialiser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'initialisers' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'linearisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'linearisations' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'parametrised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'paravirtualisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'paravirtualise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'paravirtualised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'polariser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'regulariser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'reinitialisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'reinitialises' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'reinitialising' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'standardiser' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'summarisation' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'templatised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'uninitialise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'uninitialises' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'unsynchronised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'virtualise' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'virtualised' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'virtualises' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt
E error 'virtualising' should be in aspell dictionaries (en_GB) for dictionary /home/runner/work/codespell/codespell/codespell_lib/tests/../data/dictionary_en-GB_to_en-US.txt

any ideas @lurch?

@lurch

Copy link
Copy Markdown
Contributor

I think that might be the "after #1485" part of this PR, i.e. #1485 needs to be merged, before this PR can be merged?
But that kinda goes beyond my knowledge of how codespell works. ping @peternewman

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dictionaryChanges to the dictionary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@sebweb3r@lurch@bl-ue@peternewman