Problem/Motivation

Sadly, the dictionary is out of sync again. I thought I have rebuilt the dictionary on each of the spelling issues lately but I obviously missed some.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3392428

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

quietone created an issue. See original summary.

quietone opened merge request !4963

quietone’s picture

quietone commented

The failure is unrelated, 

    1)
    Drupal\Tests\big_pipe\Unit\Render\FiberPlaceholderTest::testLongPlaceholderFiberSuspendingLoop
    Drupal\Core\DependencyInjection\ContainerNotInitializedException:
    \Drupal::$container is not initialized yet. \Drupal::setContainer() must be
    called with a real container.
quietone’s picture

quietone commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Ran spellcheck:make-drupal-dict but got 1 additional word, -behaviour. Don't want to take myself out of the ability to RTBC if that could be added though.

quietone’s picture

quietone commented
Status: Needs work » Needs review

I rebased and re-built the dictionary and there were no changes to the dictionary. @smustgrave, perhaps you were not working from HEAD?

smustgrave’s picture

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

Thought I was but if you reran and got the same good enough for me

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Closed (duplicate)

The dictionary was rebuilt in #3399123: Upgrade JavaScript dependencies to latest minor/patch releases and all four cases here are already resolved (plus more, via the upgraded cspell).

quietone closed merge request !4963