static-analysis

The below is from a conversation on #4754 regarding language values. Created a ticket per @wu-s-john 's suggestion:

What do we think of pretty printing this more like a bulleted list or zshrc autocomplete columns? I suspect it adds too much complexity for the sake of this PR, and I am largely just brainstorming how to make the UX of this CLI feel modern and be easy to read/navigate.

E

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

The stub for preg_replace_callback() is:

https://github.com/vimeo/psalm/blob/36d5a2a83c1d7b3e2aca1cfe9e9dcc916619e962/stubs/CoreGenericFunctions.phpstub#L922-L932

With @param callable(string[]):string $callback. However if you pass the PREG_UNMATCHED_AS_NULL flag the signature will become @param callable((string|null)[]):string $callback.

I have a $m[1] === null check in my call

Expected Behavior of the rule

suspend fun CoroutineScope.foo() { }

You should not use suspend and CoroutineScope. Instead use suspend fun foo() = coroutineScope.

Context

IntelliJ warning: