Go / Add additional context to error messages

[raserva]

As a follow-up, let's add a TODO to add context to these errors. It's generally a best practice to wrap all errors with some context:

return nil, fmt.Errorf("failed to do whatever: %w", err)

Originally posted by @sethvargo in #150 (comment)

The above was in reference to the following file: clients/go/pkg/auditopt/config.go however it occurs elsewhere in our go codebase as well.

[yolocs]

To be clear, the general advice is right but we should not blindly add additional context for every err return. I've seen many cases where the inner err already provides clear error message and the outer message simply repeats it. We should avoid that.

[crwilcox]

Agreed. in general more context is better, but too much of a good thing can be a bad thing.