- Issue created by @ankitv18
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 8.1 & MySQL 5.7last update
10 months ago Not currently mergeable. - last update
10 months ago 14 pass - last update
10 months ago 14 pass - Issue was unassigned.
- Status changed to Needs review
10 months ago 10:00am 14 June 2024 - 🇮🇳India ankitv18
CSpell pipeline is passing now: https://git.drupalcode.org/issue/default_content-3454594/-/jobs/1860474
Please review MR!47 - Status changed to Needs work
7 months ago 8:49pm 22 August 2024 - Status changed to Needs review
7 months ago 6:38am 23 August 2024 - 🇮🇳India ankitv18
@berdir Thanks for the suggestion ~~ I've update the MR with the changes, hence moving back into review.
- Status changed to RTBC
7 months ago 11:45am 23 August 2024 -
berdir →
committed cfb9501e on 2.0.x authored by
ankitv18 →
Issue #3454594: Fix CSpell pipeline reported in gitlab
-
berdir →
committed cfb9501e on 2.0.x authored by
ankitv18 →
- Status changed to Fixed
7 months ago 9:09pm 23 August 2024 - 🇨🇭Switzerland berdir Switzerland
Merged, but instead of ignoring drush.inc, I'd really like to drop that, it's dead code, drush 8 is long dead for D8+.
Issues/merge requests to remove that file and update ignore list would be welcome.
- 🇨🇭Switzerland berdir Switzerland
.drush.inc is now gone, but I forgot to update the cspell.
I also saw where the simplesitemap thing is coming from, see ✨ Drush comand to export all items from entity with defined criteria Needs review , that's bogus and should be fixed as done there.
Automatically closed - issue fixed for 2 weeks with no activity.