[feature request] let configuration methods return a pointer to the instance

[turtleDev]

package something

import (
    "github.com/gorilla/schema"
)

var decoder = schema.NewDecoder().IgnoreUnknownKeys(true) // valid syntax

The syntax above is valid go, and is how the docs suggest you use gorilla (keeping a global instance)

however, doing this won't be valid

package something

import (
    "github.com/gorilla/schema"
)

var decoder = schema.NewDecoder()
decoder.IgnoreUnknownKeys(true)  // syntax-error: non-declaration statement outside function body

Sounds like a simple enough change. I'd be happy to send a PR.
What do you guys think?

[elithrar]

Sure: keeping the examples accurate is important.

…

On Mon, Nov 20, 2017 at 1:18 AM turtleDev ***@***.***> wrote: package something import ( "github.com/gorilla/schema)var decoder = schema.NewDecoder().IgnoreUnknownKeys() The syntax above is valid go, and is how the docs suggest you use gorilla (keeping a global instance) however, doing this won't be valid package somethingimport ( "github.com/gorilla/schema" ) var decoder = schema.NewDecoder() decoder.IgnoreUnknownKeys() // syntax-error: non-declaration statement outside function body Sounds like a simple enough change. I'd be happy to send a PR. What do you guys think? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#93>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABIcNLiGClN3jG-Aj2YTspHi-v949RVks5s4UPpgaJpZM4Qj8zR> .

[stale]

This issue has been automatically marked as stale because it hasn't seen a recent update. It'll be automatically closed in a few days.