static-analysis

Describe the bug
Patterns for Go multiple return values match more strictly than the argument list, which is confusing.

To Reproduce
Try to match any Go function that has error in the return type list. All six of these functions should match:

func a(b int) error { return nil }
func b() (int, error) { return 0, nil }
func c() (error, int) { return nil, 0 }
func d() (float6

Many repositories need to fix, so please help if you like.

If you could help, it would be helpful if you could comment before starting the work not to overlapping.

Fix example

Run exit command after lint.

echo '::group:: Running golangci-lint with reviewdog 🐶 ...'
go

Hello I've come across an issue while constructing an array with timestamps used as keys, psalm is reporting false|int can't be used as array key while getTimestamp always returns an int.

e.g. https://psalm.dev/r/c45a9a0a0f

Expected Behavior of the rule

UnnecessaryAbstractClass should have a concise description like other rules

Context

I just updated Detekt 1.19.0 to 1.20.0 and got this failure:
https://github.com/TWiStErRob/net.twisterrob.cinema/security/code-scanning/227

The message is just plain weird to be this long:
![image](https://user-images.githubusercontent.com/2906988/164910306-4b4478c4-0