s/overriden/overridden

CommentFileSizeAuthor
#2 configuration-spelling-overridden-2079877-2.patch2.61 KBjonathan_hunt
configuration-spelling-overridden.patch1.36 KBjonathan_hunt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, configuration-spelling-overridden.patch, failed testing.

jonathan_hunt’s picture

jonathan_hunt CreditAttribution: jonathan_hunt commented
Status: Needs work » Needs review
FileSize
configuration-spelling-overridden-2079877-2.patch2.61 KB

Missed a couple of instances.

Status: Needs review » Needs work

The last submitted patch, configuration-spelling-overridden-2079877-2.patch, failed testing.

lucascaro’s picture

lucascaro CreditAttribution: lucascaro commented
Status: Needs work » Needs review

The test that failed here is failing on 7.x-2.x so this might be actually good to go...

star-szr’s picture

star-szr
he/him
Primary language English
CreditAttribution: star-szr commented
Version: 7.x-2.0-alpha3 » 7.x-2.x-dev
Status: Needs review » Reviewed & tested by the community

The patch applies fine, although the issue should be against 7.x-2.x-dev (which is currently failing tests, so this patch will not pass until HEAD is fixed).

Thanks @jonathan_hunt :)

benjy’s picture

benjy CreditAttribution: benjy commented

2: configuration-spelling-overridden-2079877-2.patch queued for re-testing.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
CreditAttribution: dagmar commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed and Pushed to 7.x-2.x. Thanks.

Status: Fixed » Closed (fixed)

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