search

taylor / kiex

Elixir version manager
http://taylor.github.io/kiex/
MIT License
766 stars 65 forks source link

Fix broken `list branches` functionality #6

Closed bitwalker closed 10 years ago

bitwalker commented 10 years ago

So the JSON result for fetching the branches is pretty complex. I tried a couple different ways of slicing and dicing it, but my bash-fu isn't strong enough. So instead I deferred to the usage of python -mtool.json. This should be included in any of the supported distributions, but I can certainly understand if you don't want a dependency on that being available. That said, it sure makes this a lot more painless, and now it works too, so there's that.

Anyways, I'm going to be using this, so figured I'd send this change your way.

taylor commented 10 years ago

Created a fix with bash. I love the different json tools, but you are correct that I am avoiding anything non-bash.

This was a regression when I was adding caching and things broke with the new json output for the github api.

I really need some command line testing :) (probably with Cucumber and aruba)

bitwalker commented 10 years ago

Right on, thanks for the quick response!