Problem/Motivation

Discovered at #2972224: Add .cspell.json to automate spellchecking in Drupal core

./core/modules/system/tests/src/Functional/Theme/ThemeUiTest.php has 5 references to "incompatitable" (only in 9.0.x branch and up).

Either @tedbow or myself have introduced and fixed this typo many times in the last few months. ;) Feeling personally responsible to help resolve it.

Proposed resolution

s/incompatitable/incompatible/

Remaining tasks

  1. Do eet.
  2. Review.
  3. RTBC.
  4. Commit.
  5. (Perhaps) Remove this from core/.cspell.json if #2972224: Add .cspell.json to automate spellchecking in Drupal core has landed by then.

User interface changes

None - typo only in test code.

API changes

None.

Data model changes

Nope.

Release notes snippet

Nada.

CommentFileSizeAuthor
#2 3138671-2.patch2.53 KBdww

Comments

dww created an issue. See original summary.

dww’s picture

dww
we/he/they
commented
Assigned: dww » Unassigned
Issue summary: View changes
Status: Active » Needs review
Related issues: +#2972224: Add .cspell.json to automate spellchecking in Drupal core
StatusFileSize
new 3138671-2.patch2.53 KB
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Let's get this fixed!

  • xjm committed 9e0ed28 on 9.1.x 
    Issue #3138671 by dww: Fix "incompatitable" typos in core

  • xjm committed 074dbea on 9.0.x 
    Issue #3138671 by dww: Fix "incompatitable" typos in core

(cherry...
xjm’s picture

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

Confirmed this is the last of 'em; grep -ri "incompatitable" * returns no results.

Since this is just internal fixes in the one test (we caught the rest of them before commit), it's backportable as far as it will go. Interestingly, the test in 8.9 already is free from this misspellling. Not sure how that happened, but marking fixed against 9.0.x.

Thanks!

Status: Fixed » Closed (fixed)

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