Add support for local cspell tests

added Doing label

assigned to @jurgenhaas

mentioned in commit 1f2f840e

There is now a command ahoy test cspellmodule [MODULENAME] which performs the same cspell tests as gitlab@drupal, just locally.

added Documentation label and removed Doing label

unassigned @jurgenhaas

mentioned in merge request !19 (merged)

After execution on a latest 10.3 Drupal installation:

a test cspellmodule amos_sync

I get the following:

cspellmodule: line 17: /drupal/web/core/node_modules/.bin/cspell: No such file or directory
cspellmodule: line 19: /drupal/web/core/node_modules/.bin/cspell: No such file or directory

If I miss something, I can put it into the documentation.

assigned to @jurgenhaas

@danielspeicher there is ahoy test stylelintprepare which prepares all of core's yarn packages that are required for development. That needs to be called once, so that all the packages get prepared in the local installation. They remain in place until core gets updated, then this needs to be called once again.

The name stylelintprepare is not ideal, I see. It has been chosen when this was implemented as a preparation for the ahoy test stylelintmodule feature some time before this one. Back then, I didn't realize that the same preparation was required not only for style lint but also for cspell, and maybe others in the future as well.

Shall we rename the ahoy test stylelintprepare into ahoy test preparecorefordev or something like that?

assigned to @danielspeicher and unassigned @jurgenhaas

@jurgenhaas Ok, great. That worked.

Yes, I think we should rename it in ahoy test preparecorefordev. Makes it even more clear.

assigned to @jurgenhaas and unassigned @danielspeicher

mentioned in commit fffd594f

@danielspeicher I've renamed that task, see commit fffd594f

assigned to @danielspeicher and unassigned @jurgenhaas

mentioned in commit devops-tools/documentation@c7652680

closed

mentioned in merge request devops-tools/documentation!20 (merged)

mentioned in commit devops-tools/documentation@eb13ab76

mentioned in commit 5f3b74b7

mentioned in merge request !20 (merged)