I'm working on a number of cleanup's..

Partly split off from: #2014519: SQLite test fails due to missing path while restoring database

I'll first test it using a pull request, then prepare a full patch.

CommentFileSizeAuthor
#5 drush.miscellaneous.2034731-5.patch10.97 KBhelmo
#3 drush.miscellaneous.2034731-3.patch25.73 KBhelmo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helmo’s picture

helmo CreditAttribution: helmo commented

PR added: https://github.com/helmo/drush/pull/3

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Active » Needs work

Until we switch to github, it would be easiest if you posted patches here. Thanks.

helmo’s picture

helmo CreditAttribution: helmo commented
Status: Needs work » Needs review
FileSize
drush.miscellaneous.2034731-3.patch25.73 KB

Sure here it is.

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Needs review » Fixed

committed

helmo’s picture

helmo CreditAttribution: helmo commented
Status: Fixed » Reviewed & tested by the community
FileSize
drush.miscellaneous.2034731-5.patch10.97 KB

It seems that you only committed the changes to the tests directory.

moshe weitzman’s picture

moshe weitzman CreditAttribution: moshe weitzman commented
Status: Reviewed & tested by the community » Fixed

Committed. Not sure how that happenned.

Status: Fixed » Closed (fixed)

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