ajvb / kala

Modern Job Scheduler
MIT License
2.13k stars 187 forks source link

remove sling from client #126

Closed jlburkhead closed 7 years ago

jlburkhead commented 8 years ago

This removes the dependency on github.com/dghubble/sling from the kala client (https://github.com/ajvb/kala/issues/45).

jlburkhead commented 8 years ago

might be time to upgrade the go version

ajvb commented 8 years ago

@jlburkhead I'd like to be able to support older versions of Go, so lets define the relevant MethodX consts in this file.

ajvb commented 8 years ago

Excluding the already existing comments, this looks great! Thanks so much for the contribution @jlburkhead

coveralls commented 8 years ago

Coverage Status

Coverage increased (+1.0%) to 81.35% when pulling 88d84b41ad94d3a6d99208c0cf1cc7ac0c979e3e on jlburkhead:remove_sling into cd5ec293a21d527aa6a745df62f6c1e8f9a3fbcb on ajvb:master.