search

500px / gunter

Language agnostic task wrapper and loyal servant.
MIT License
9 stars 1 forks source link

Merge localTask and remoteTask #7

Closed devonoel closed 9 years ago

devonoel commented 9 years ago

localTask and remoteTask represent a pretty obnoxious code smell in the library. There's both a lot of repetition between the two, and a lot of weird subtle differences between how they work, such as in issue #6.

Ideally, shelljs and sequest should both be removed in favour of a library that can handle the needs of both localTask and remoteTask, and these two methods should be merged.

devonoel commented 9 years ago

This has been substantially improved by removing sequest in favour of ssh2, and through some other enhancements. For now, I'll close this issue, although it may be useful to eventually extract this behaviour into its own library.