- Issue created by @spokje
- π³π±Netherlands spokje
Mirroring status of parent issue, which became critical.
- π³π±Netherlands spokje
spokje β changed the visibility of the branch 11.x to hidden.
- π³π±Netherlands spokje
spokje β changed the visibility of the branch 3487908- to hidden.
- πΊπΈUnited States smustgrave
I applied the MR for 11.x and the cspell commands appear to work as expected. Assuming the word changes are either the file out of sync or maybe cspell updated their internal dictionaries?
- π³π±Netherlands spokje
Assuming the word changes are either the file out of sync or maybe cspell updated their internal dictionaries?
The latter for sure, that's the main part of each new minor bump in
cspell
.The first part: No clue, maybe, maybe not.
The Needs Review Queue Bot β tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily 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.
- π³π±Netherlands spokje
Why don't you help yourself to a nice cold glass of ST[Bleep!]U, needs-review-queue-bot?
- πΊπΈUnited States smustgrave
Hey the bot helps me keep the queue low hahaha
But I do think it got confused by the multiple MRs on this one.
- π³π±Netherlands spokje
Combo of multiple MRs and it isn't too clever when it comes to *.lock files IMHO, both composer.lock and yarn.lock.
Anyway, this issue was bumped and you noticed it because of the bot, so that's good enough for me :)
-
longwave β
committed d2cfad9c on 11.1.x
Issue #3487908 by spokje, smustgrave: Update cspell to latest
-
longwave β
committed d2cfad9c on 11.1.x
-
longwave β
committed 04478f36 on 11.x
Issue #3487908 by spokje, smustgrave: Update cspell to latest
-
longwave β
committed 04478f36 on 11.x
- π¬π§United Kingdom longwave UK
Committed and pushed to 11.x and 11.1.x, thanks!
10.5.x and 10.4.x don't apply, looks like a merge conflict with another JS dependency update, moving to NW for that.
- First commit to issue fork.
- π³πΏNew Zealand quietone
Rebased the 10.4 MR and started over for the 10.5 one. No reason for that, it is just how I did it between other things at home.
- π³π±Netherlands spokje
Unsure if I, as the creator of the original branches can RTBC, but since there's not much time left before 10.4.0-RC is shipped, I'm willing to bend the rules and RTBC-ed.
#yoloThanks @quietone.
-
longwave β
committed bb51f1da on 10.4.x
Issue #3487908 by spokje, quietone, smustgrave: Update cspell to latest
-
longwave β
committed bb51f1da on 10.4.x
-
longwave β
committed 9271d92b on 10.5.x
Issue #3487908 by spokje, quietone, smustgrave: Update cspell to latest
-
longwave β
committed 9271d92b on 10.5.x
- π¬π§United Kingdom longwave UK
Committed and pushed to 10.5.x and 10.4.x, thanks!
- First commit to issue fork.
Automatically closed - issue fixed for 2 weeks with no activity.