Add "funcresult" linter

[leonklingele]

https://github.com/leonklingele/funcresult

[ldez]

Hello,

we already have a linter that handles those cases: https://go-critic.com/overview#unnamedresult

[leonklingele]

@ldez it's not the same. The funcresult linter unconditionally enforces the use of either named or unnamed result parameters, while unnamedResult only conditionally decides whether result params should be named.

[ldez]

For me, it's something that can be added to the existing linter.

[leonklingele]

I've opened go-critic/go-critic#1201. Let's see if I can integrate it to go-critic.

[leonklingele]

seeing that this was only closed due to the use of the require-named option, I've removed support for it from this PR.
@ldez can you please reopen?

[ldez]

seeing that this was only closed due to the use of the require-named option,

It's not exactly what I say: #2701 (comment)

PR #2701 already provides the feature, so I will not reopen this one.

OSS is about sharing, neither authorship nor ownership, I recommend contributing on the linter added in #2701.