Fix spelling of "email"

I tried again today to use cspell for this. I added "e-mail" to the flagWords and ran spellcheck on core and the instances were found. So, something has changed since longwave tried this in #17.

A core committer explained in #8 that work was not needed until a sniff was made in Coder. And that was repeated in #15 and #17. Patches and screenshots continued to be posted. That is not helping to resolve this and make noise in the issue. I am removing credit per How is credit granted for Drupal core issues → .

Attached patch against Drupal 10.1.x

Let's get this back on track.

I have made a new patch, from scratch. I added 'e-mail' as a flag word and the spell checked all of core to find the instances. I fixed those and spell checked again. The dictionary does not need to be changed. I haven't included an interdiff because the previous patches include changes to migrate related files and that will just add noise to the interdiff.

The reroll in #36 does not have a diff and it didn't take into account the previous comments about the change of approach nor that the changes to migrate files (#8) should be removed. Therefor, I am removing credit.

Ran yarn spellcheck:core locally and it didn't return any errors:

CSpell: Files checked: 14818, Issues found: 0 in 0 files

Reviewed the patch manually to confirm all of the changes there look good 👍

+++ b/core/.cspell.json
@@ -49,7 +49,8 @@
     "flagWords": [
       "grey",
       "hte",
-      "please"
+      "please",
+      "e-mail"
     ],

Nitpick: should we keep this list in alphabetical order?

Yes, they should

Interdiff looks good 👍

The last submitted patch, 42: 3051548-42.patch, failed testing. View results →

Random fail: 🐛 Random test fail in Drupal\Tests\Component\Utility\RandomTest::testRandomMachineNamesUniqueness Needs work .

Committed 758cc0e and pushed to 11.x. Thanks!

Credit on this seems to have been lost, re-adding.

Automatically closed - issue fixed for 2 weeks with no activity.

As this has been fixed without the use of a new sniff in Coder, it is confusing to have that issue linked in the IS, therefore I have removed it. It is still in the related issues, which is OK.