Problem/Motivation

CSpell pipeline: https://git.drupalcode.org/project/default_content/-/jobs/1856157

Issue fork default_content-3454594

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:

About issue forks

Comments

ankitv18 created an issue. See original summary.

ankitv18 opened merge request !47

ankitv18’s picture

ankitv18 commented
Assigned: ankitv18 » Unassigned
Status: Active » Needs review

CSpell pipeline is passing now: https://git.drupalcode.org/issue/default_content-3454594/-/jobs/1860474
Please review MR!47

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Needs work
ankitv18’s picture

ankitv18 commented
Status: Needs work » Needs review

@berdir Thanks for the suggestion ~~ I've update the MR with the changes, hence moving back into review.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Looks ready

  • berdir committed cfb9501e on 2.0.x authored by ankitv18 
    Issue #3454594: Fix CSpell pipeline reported in gitlab
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

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.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

.drush.inc is now gone, but I forgot to update the cspell.

I also saw where the simplesitemap thing is coming from, see #2786479: Drush comand to export all items from entity with defined criteria , that's bogus and should be fixed as done there.

Status: Fixed » Closed (fixed)

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