Jonas Huckestein jonashuckestein

jonashuckestein commented on pull request mondough/typhon#39
@jonashuckestein

I'll merge this tomorrow once we've merged a few of those PRs

jonashuckestein commented on pull request mondough/typhon#39
@jonashuckestein

LGTM :boom:

jonashuckestein deleted branch upstream-update at mondough/mercury
@jonashuckestein
  • @jonashuckestein b7d0023
    Merge pull request #3 from mondough/upstream-update
  • @obeattie d550e16
    Merge remote-tracking branch 'obeattie/master' into upstream-update
  • 4 more commits »
jonashuckestein merged pull request mondough/mercury#3
@jonashuckestein
Marshaler registry
5 commits with 138 additions and 90 deletions
jonashuckestein commented on pull request mondough/mercury#3
@jonashuckestein

LGTM. One tiny thing that I'm happy to add in a different PR

jonashuckestein commented on pull request mondough/mercury#3
@jonashuckestein

Would love to attach the panic information to the error somehow. May be worth having terrors.Panic in the future

@jonashuckestein
jonashuckestein opened pull request mondough/typhon#38
@jonashuckestein
Nifty err.FullString() method
1 commit with 5 additions and 0 deletions
@jonashuckestein
  • @jonashuckestein ec6632c
    publish single docker image
@jonashuckestein
  • @jonashuckestein 51376ab
    Update to cassandra 2.1.8
jonashuckestein commented on pull request mondough/mercury#2
@jonashuckestein

Sorry, wasn't paying attention

@jonashuckestein
  • @jonashuckestein 485ea4c
    why construct the response twice?
jonashuckestein opened pull request mondough/mercury#2
@jonashuckestein
Verbosely log errors in mercury handlers
1 commit with 4 additions and 1 deletion
jonashuckestein deleted branch update at mondough/gocql
@jonashuckestein
jonashuckestein merged pull request mondough/gocql#2
@jonashuckestein
Update from upstream
198 commits with 5,871 additions and 1,519 deletions
jonashuckestein commented on pull request mondough/gocql#2
@jonashuckestein

wearedev acceptance tests pass when using this branch. Will just go ahead and merge it :boom:

jonashuckestein deleted branch upstream-update at mondough/mercury
@jonashuckestein
  • @jonashuckestein f427584
    Merge pull request #1 from mondough/upstream-update
  • @obeattie 260e5d6
    Merge remote-tracking branch 'obeattie/master' into upstream-update
  • 2 more commits »
jonashuckestein merged pull request mondough/mercury#1
@jonashuckestein
Fix JSON error deserialisation
3 commits with 48 additions and 2 deletions
jonashuckestein commented on pull request mondough/mercury#1
@jonashuckestein

LGTM, sorry for the delay!

jonashuckestein deleted branch update-upstream at mondough/gocassa
@jonashuckestein
jonashuckestein merged pull request mondough/gocassa#5
@jonashuckestein
Update from upstream
52 commits with 1,472 additions and 416 deletions