Fix argument testing in homeshick.fish

andsens / homeshick

git dotfiles synchronizer written in bash

MIT License

2.11k stars 145 forks source link

Fix argument testing in homeshick.fish #175

Closed psliwka closed 6 years ago

psliwka commented 6 years ago

Hi, just noticed a small bug in the fish wrapper, which causes it to print error message when invoked without arguments. This PR adds an unit test, and fixes the issue.

Thanks for this cool project BTW!

andsens commented 6 years ago

@herrbischoff, could you sign off on this? Seems OK to me, but I am on vacation right now, hence the slow response time and the inability to test :-)

herrbischoff commented 6 years ago

LGTM 👍

andsens commented 6 years ago

Cool! Merging.

Thank you @psliwka!