Anton Telyshev

Sometimes it's ok to ignore `recover()` without explicit `_` and it doesn't matter since `recover()` is just a special case of a more general case. Some examples from std: ```go...

+1 ``` $ go get gopkg.in/russross/blackfriday.v2 go: gopkg.in/russross/blackfriday.v2 upgrade => v2.0.1 go get: gopkg.in/russross/[email protected]: parsing go.mod: module declares its path as: github.com/russross/blackfriday/v2 but was required as: gopkg.in/russross/blackfriday.v2 ```

+1 for map preallocation checking

@robdefeo why is [assert.ErrorIs](https://pkg.go.dev/github.com/stretchr/[email protected]/assert#ErrorIs) not enough?

I see. Your tests written in bad pattern: it's better to just check for an error (how was before) than to use its text for comparison. `EqualError`, `ErrorContains` are examples...

+1

> @Antonboom Does it look mergeable now? @feldgendler, I'm not the owner of repo :( I'm also interested in this fix and was just passing by. @achille-roussel, merge please...

+1

> That code in the latest version will generate that body parameter if the spec has a body, which it determines by checking for spec.RequestBody It's for client only, not...

+1