Josh Bleecher Snyder

Results 323 comments of Josh Bleecher Snyder
trafficstars

I implemented WriteOpts as discussed here, and renamed Writeable to WriterTo. I think this has helped. I'm going to leave this open for future discussion and work on the original...

> Here's another proposal - make Writeable half of what it is. This interacts with https://github.com/pkg/diff/issues/4, which would get rid of the `Filename` part of `Writeable`. At which point it...

I just pushed some commits that make this a bit cleaner. I still agree it'd be nice to do something cleaner here; writing is a bit of a fly in...

I have had a similar idea. Quoting from diff.go: ```go // TODO: consider adding an "it just works" test helper that accepts two slices (via interface{}), // diffs them using...

Hmmm. It may have done. (I've been writing and tweaking code since, I'm afraid.)

@protocool yep, I see the same thing, with git version `1.7.11.2`. That's gotta be it.

I switched to https://github.com/rowanj/gitx, and highly recommend it. And yes, it handles submodules well.

For type switches, if you have `type T int | interface{}` and you do: ``` switch t := t.(type) { case int: // ... ``` and t contains an interface{}...