karlfreeman / middleman-deploy

Deploy a middleman built site over rsync, ftp, stfp, or git (e.g. gh-pages on github).
MIT License
553 stars 124 forks source link

If deploy.user is set, use it also in rsync command #113

Open nlsrchtr opened 8 years ago

nlsrchtr commented 8 years ago

This PR fixes a small bug in the rsync strategy. If the user was explicitly set, it wasn't passed into the rsync command, due to local / global method / variable name clashes.

wilsonmichael commented 8 years ago

This fixed this issue for me, hope this pull request is accepted soon.

gavinbaradic commented 8 years ago

+1

klacointe commented 8 years ago

:+1:

tonytonyjan commented 8 years ago

thanks!

jugutier commented 7 years ago

+1