My IDE is set to remove any trailing space per Drupal coding standards. Creating a patch for drush is annoying when you end up with a monster patch for just a few code changes :( So for rolling a patch I have to turn off trailing spaces striping on my IDE, revert all changes and reapply the relevant ones one by one.

CommentFileSizeAuthor
#4 638324_codestyle_4.patch43.53 KBscor
strip_trail_spaces.drush_.patch104.97 KBscor

Comments

axyjo’s picture

axyjo commented
Status: Active » Needs review

Looks good.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

Committed. A few hunks would no longer apply so those are left as is. Feel free to resubmit so we nail those.

Status: Fixed » Closed (fixed)

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

scor’s picture

scor commented
Status: Closed (fixed) » Needs review
StatusFileSize
new 638324_codestyle_4.patch43.53 KB

nailing more of these down, including some missing new line at end of files. (see script)

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed. thanks.

Status: Fixed » Closed (fixed)

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