- First commit to issue fork.
- @longwave opened merge request.
- 🇬🇧United Kingdom longwave UK
Removed all cspell:ignore lines with
rg -l cspell:ignore|xargs sed -i '/cspell:ignore/d'
Let's see what happens.
- Status changed to Needs review
about 1 year ago 12:12pm 13 November 2023 - 🇬🇧United Kingdom longwave UK
Postponed issue was closed as won't fix so let's just make this active again.
- 🇬🇧United Kingdom longwave UK
I do wonder if we should enable the
allowCompoundWords
setting which would allow words like "objectindex", or maybe on a per-file basis? - Status changed to Needs work
about 1 year ago 12:25pm 13 November 2023 - 🇬🇧United Kingdom longwave UK
The results are in: https://git.drupalcode.org/project/drupal/-/jobs/321950
- Status changed to Needs review
about 1 year ago 12:51pm 13 November 2023 - 🇳🇿New Zealand quietone
There is 📌 Test for words in 'cspell:ignore' that can be removed Needs work which is adding a script to check that all words in a
cspell:ignore
line are still needed. I think this is a duplicate of that, or am I mistaken? - Status changed to Needs work
about 1 year ago 1:30pm 13 November 2023 The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- Status changed to Closed: duplicate
about 1 year ago 1:37pm 13 November 2023 - 🇬🇧United Kingdom longwave UK
+1, wasn't following that other issue, closing this as duplicate.