Skip to content

Draft: clean out some typos

Josh Soref requested to merge issue/drupal-3122088:3122088-spelling into 9.5.x

I was testing my own tooling which has significant overlap to cspell (including the ability to suggest/use its dictionaries).

I've written some patterns that help me ignore content: https://github.com/check-spelling/drupal/blob/d1d4b216cfa55d593987b4f143ab6313508a2eef/.github/actions/spelling/patterns.txt And selected patterns for ignoring files entirely: https://github.com/check-spelling/drupal/blob/d1d4b216cfa55d593987b4f143ab6313508a2eef/.github/actions/spelling/excludes.txt

I'm happy to drop any/all of the changes, or to split anything into distinct issues. The Drupal advice said to search for an existing issue before filing one, and this issue is definitely related -- my starting point was indeed looking at the cspell dictionary file and identifying things that weren't really words.

The drupal guidelines say to prefer en-US, so in a few rare cases, I did switch things (mostly because they were already really in the minority in the codebase).

I have some extra rules against specific additional patterns (e.g. non-existent or non existent or not existent), but if the project doesn't want those changes, I'm happy to drop them as well: https://github.com/check-spelling/drupal/blob/d1d4b216cfa55d593987b4f143ab6313508a2eef/.github/actions/spelling/line_forbidden.patterns

For perspective, to the main ticket, the current dictionary has 1679 entries, whereas I've promoted ~1200 of them into the dictionary I'm using and have a bunch of items I don't think are words with ~1200 entries.

Note that I started my work on 10.1.x and this is a backport, which means that it's possible a bunch of items I've added to the dictionary aren't present, but I treat my dictionary as "words that are words whether or not they're currently used" whereas "expect" is "tokens that were seen but not found in the dictionary").

Merge request reports

Loading