Gitlab templates has an enhancement to automatically add the Lorum Ipsum dictionary to cSpell, to save maintainers the trouble of having to ignore all of this. See #3436206: Add lorem-ipsum spelling dictionary

The work is ready to be tested, and can be done by adding the following into .gitlab-ci.yml

  # MR 165 Add lorem-ipsum dictionary
  - project: issue/gitlab_templates-3436206
    ref: 3436206-lorem-ipsum-dictionary

variables:
  _CURL_TEMPLATES_REPO: issue/gitlab_templates-3436206
  _CURL_TEMPLATES_REF: 3436206-lorem-ipsum-dictionary
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:

Comments

jonathan1055 created an issue. See original summary.

jonathan1055’s picture

First commit removes the cspell:disable-next-line from seven files that have the "Lorem ipsum" lines. The cspell job fails as expected.
https://git.drupalcode.org/issue/ui_suite_uswds-3436424/-/pipelines/130459

CSpell: Files checked: 316, Issues found: 316 in 7 files
The number of unrecognised/misspelled words is 67
jonathan1055’s picture

Issue summary: View changes

Now a pipeline using gitlab_templates MR165
https://git.drupalcode.org/issue/ui_suite_uswds-3436424/-/pipelines/130466

Interestingly this does not pass as expected. All of the words are OK except one

./templates/patterns/process_list/process_list.ui_patterns.yml:27:57
- Unknown word (consectetuer) -- ipsum dolor sit amet, consectetuer adipiscing elit.
Suggestions: [consectetur]
...
CSpell: Files checked: 316, Issues found: 5 in 3 files
The number of unrecognised/misspelled words is 1
=== This is _cspell_unrecognized_words.txt ===
consectetuer

The suggestion for replacement is "consectetur" which is obviously the word in the lorum-ipsum dictionary. This word is already used and spelled correctly in four of the files, but wrong in the other three.

jonathan1055’s picture

Status: Active » Needs review

I fixed the spelling in the three files, I guess it was a copy/paste from one original that was wrong somehow. The job passes green.
https://git.drupalcode.org/issue/ui_suite_uswds-3436424/-/pipelines/130484

When the gitlab_templates MR has been merged and then rolled out to all contrib on the 'default-ref' branch then this MR can be committed. The temporary change to .gitlab-ci.yml here will need to be removed before merging.

smustgrave’s picture

Status: Needs review » Fixed

That was a fast fix!

jonathan1055’s picture

Just be aware that with the above commit you are now using a gitlab_templates version which is currently slight ahead of the 'default-ref' but is also static. In due course you will want to revert just the .gilab-ci.yml change to put back

- project: $_GITLAB_TEMPLATES_REPO
  ref: $_GITLAB_TEMPLATES_REF

which will use the recommended 'default-ref', and delete the two $_CURL variables. There is no hurry, as the MR still exists and will be usable, but you will not be getting any of the newer work when 'default-ref' gets updated.

When #3436206: Add lorem-ipsum spelling dictionary is merged you will see the commit listed in this link: compare/default-ref...main?from_project_id=96292
This shows all commits that are in 'main' but not yet reached 'default-ref'. If the commit is showing here then you need to wait.

You can also see all the commits and where the tags are. This currently shows three commits which are above the set of tags, meaning that these are only in 'main' and have not yet reached 'default-ref'

Status: Fixed » Closed (fixed)

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

jonathan1055’s picture