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 @@
+      "compatibilty",
+      "compatiblity",

Proposed resolution

@xjm: compatibility.

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

CommentFileSizeAuthor
#12 3138792-12-8.8.x.patch539 bytessja112
#3 3138792-3.patch1.22 KBsja112

Comments

jungle created an issue. See original summary.

sja112’s picture

sja112 commented
Assigned: Unassigned » sja112
sja112’s picture

sja112 commented
Assigned: sja112 » Unassigned
Status: Active » Needs review
StatusFileSize
new 3138792-3.patch1.22 KB
dww’s picture

dww
we/he/they
commented
Status: Needs review » Reviewed & tested by the community

Confirmed these are the only two (remaining) instances of either of the misspellings mentioned in the summary.

Bot is basically happy. There's a false-positive CS complaint:

/var/lib/drupalci/workspace/jenkins-drupal_patches-46457/source/core/misc/jquery.cookie.shim.es6.js
line 0	File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

but that's not a real problem.

The generated core/misc/jquery.cookie.shim.js file has no comments, so the typo is not present there and we don't want/need to touch that file with this patch.

Therefore, RTBC. :)

Thanks,
-Derek

p.s. I was fairly certain some of my typos would be in this issue when I saw it on the list, but I guess we already fixed those in other places like #3138671: Fix "incompatitable" typos in core. /shrug

  • xjm committed 0cdcc80 on 9.1.x 
    Issue #3138792 by sja112, dww: Fix "compatibility" relevant typos in...

  • xjm committed a46b823 on 9.0.x 
    Issue #3138792 by sja112, dww: Fix "compatibility" relevant typos in...

  • xjm committed fd9bdc1 on 8.9.x 
    Issue #3138792 by sja112, dww: Fix "compatibility" relevant typos in...
xjm’s picture

xjm
she/her
Primary language English
commented
Version: 9.1.x-dev » 8.8.x-dev
Status: Reviewed & tested by the community » Fixed

Committed to 9.1.x, and backported to 9.0.x, 8.9.x, and 8.8.x as a patch-eligible documentation bugfix. Thanks!

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Fixed » Patch (to be ported)

Scratch that last bit; it doesn't cherry-pick cleanly to 8.8.x. Can we get a backport? It might just be one of the two lines.

xjm’s picture

xjm
she/her
Primary language English
commented
Title: Fix "compatibility" relevant typos in core » [backport] Fix "compatibility" relevant typos in core
sja112’s picture

sja112 commented
Assigned: Unassigned » sja112
sja112’s picture

sja112 commented
Assigned: sja112 » Unassigned
Status: Patch (to be ported) » Needs review
StatusFileSize
new 3138792-12-8.8.x.patch539 bytes

Patch for 8.8 branch.

maithri shetty’s picture

maithri shetty commented
Status: Needs review » Reviewed & tested by the community

  • xjm committed 84e5f08 on 8.8.x 
    Issue #3138792 by sja112, xjm, dww: Fix "compatibility" relevant typos...
xjm’s picture

xjm
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed the backport to 8.8.x. Thanks!

xjm’s picture

xjm
she/her
Primary language English
commented
Title: [backport] Fix "compatibility" relevant typos in core » Fix "compatibility" relevant typos in core

Status: Fixed » Closed (fixed)

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