Rick Beton

An improved adapter function is of course possible ```go func toDecimal(t Result) decimal.Decimal { switch t.Type { default: return decimal.Decimal{} case True: return decimal.NewFromInt(1) case String: n, _ := decimal.NewFromString(t.Str)...

It has occurred to me that this should only be a v2 change and that you could go further: for v2 you _don't need_ the `Result.Num` field. Removing `Num` would...

Is this effectively a duplicate of #158 ? The `Parse` method takes a `string` and the `String()` method returns a string, so the API appears to me to be sufficient....

I'm new to GJSON. I had the same idea - being able to parse streaming data can be quite an advantage for low memory footprint. But when I looked at...

I don't think the (current) embedded Equal method will work correctly. Take a look at https://play.golang.org/p/N7s_HoBc5ce, which does not compile although it looks like it ought to. The receiver and...

That probably means the Equal should be split into a separate PR.

I edited the release notes a little - no implied criticism, I was working on the basis that two minds do a better job than one.

I'm wondering about the decision not to support `0000-00-00T00:00:00`. If it might be received in JSON, the behaviour I would expect would be to parse it cleanly as zero `time.Time{}`...

`ForceApi` could include a `*http.Client` value that is initialised in `Create` to `http.DefaultClient`. ```go type ForceApi struct { ... existing httpClient *http.Client } ``` Then change `func (forceApi *ForceApi) request(...)...

btw this is a good idea, IMO.