Quinn Slack
Quinn Slack
Great. Thanks for the CR and the other fixes. I'll push some changes to this PR to address these issues in a couple of hours.
OK, I pushed changes that address these comments and bring this branch up-to-date with master. Travis CI is now passing. Regarding `resourcePaths()`, I personally find it simpler to be explicit...
@nathany, yes, good point, we will need to update the README to describe the `-d` option, but I want to wait until it's decided whether we use `-d` or the...
Sounds like buckler won't be used by shields.io anymore, so this may be a moot point, right? (Re: https://github.com/badges/shields/issues/90#issuecomment-31803963)
We've been using this branch for a few months to generate the badges at https://sourcegraph.com/help/authors/badges.
Yes, absolutely. The wire protocol is simple but only defined in Go right now. It basically sends JSON-encoded [`collectPacket`](https://sourcegraph.com/sourcegraph/apptrace@master/.GoPackage/sourcegraph.com/sourcegraph/apptrace/.def/collectPacket) structs over a TCP connection (with or without TLS). I think...
Great. We'll add this to the TODO list and will update here on this issue. I was thinking protobufs because it's not immediately clear to non-Go folks what ``` //...
Thanks, I think we will be able to do the protobuf-ization soon. If you wanted to just write a client that sent JSON and start using it now (to work...
@keegancsmith The migration sounds good to me.
[Sourcegraph](https://sourcegraph.com) uses the [Go remote import path spec](https://golang.org/cmd/go/#hdr-Remote_import_paths) to map from paths to `(git URL, commit, subdir)`. It's not universal, and it has ambiguity, so it's not a solution. To...