Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Discovered at #2972224: Add .cspell.json to automate spellchecking in Drupal core, and pointed by @xjm in https://www.drupal.org/project/drupal/issues/3122088#comment-13628724
+++ b/core/.cspell.json
@@ -0,0 +1,1288 @@
+ "Piccasso",
Proposed resolution
@xjm: Picasso.
As title and see the change record https://www.drupal.org/node/3122084 for how to work with cspell.
Remaining tasks
Pick out all applicable words from #2972224: Add .cspell.json to automate spellchecking in Drupal core and fix them.
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#5 | before_patch.png | 258.65 KB | ankitsingh0188 |
#5 | after_patch_1.png | 259.13 KB | ankitsingh0188 |
#5 | after_patch.png | 44.55 KB | ankitsingh0188 |
#4 | 3138785-3.patch | 2.67 KB | sja112 |
Comments
Comment #2
jungleComment #3
sja112 CreditAttribution: sja112 at Srijan | A Material+ Company for Drupal India Association commentedComment #4
sja112 CreditAttribution: sja112 at Srijan | A Material+ Company for Drupal India Association commentedComment #5
ankitsingh0188#4 patch looks good to me, I checked with grep -r "Piccasso" * command and there is no other word after applying the patch.
Good to move to RTBC
Comment #6
mohrerao CreditAttribution: mohrerao as a volunteer and at Srijan | A Material+ Company for Drupal India Association commentedMoving to RTBC as per @ankit's comments.
Comment #11
xjmThanks @ankit.singh for reviewing. No need to attach screenshots of your CLI; in this case, copying and pasting the commands and their output would be sufficient and also more readable.
For future reference, I'd suggest doing
grep -ri
for these types of issues to make sure there isn't a lowercase (or all-caps, etc.) variant that was missed from the original list. That'ts what I did myself when committing it just now.Committed to 9.1.x, and cherry-picked to 9.0.x, 8.9.x, and 8.8.x as a patch-eligible cleanup to test data. Thanks!