Docs: Fix return_value() inline documentation

[rollybueno]

Corrected 2 inline docs for return_value():

1. 3rd param is $values not a redundant $type.
2. It's also an array since we loop the variable and looking for $value['name'] & $value['type'].
3. @return is mixed since we are either return WP_CLI::error(), which technically null as per https://make.wordpress.org/cli/handbook/references/internal-api/wp-cli-error/, and $result[0]['value'] which is either boolean or string from WP config variables

[swissspidy]

Can we make the type more precise? I think this one is correct

Suggested change

	* @param array $values
	* @param array<array{name: string, value: string, type: string}> $values

[rollybueno]

value: mixed is more precise since it can be integer, string or boleean. e.g. define( 'WP_DEBUG', true ), define( 'SITE_ID_CURRENT_SITE', 1 ), define( 'DB_NAME', 'wp_cli_test' )

[swissspidy]

3. @return is mixed since we are either return WP_CLI::error(), which technically null as per make.wordpress.org/cli/handbook/references/internal-api/wp-cli-error, and $result[0]['value'] which is either boolean or string from WP config variables

I don't think they can be booleans.. Can you double check that?

In any case, let's try to make the types as precise as possible. string|never is more accurate

[rollybueno]

3. @return is mixed since we are either return WP_CLI::error(), which technically null as per make.wordpress.org/cli/handbook/references/internal-api/wp-cli-error, and $result[0]['value'] which is either boolean or string from WP config variables

I don't think they can be booleans.. Can you double check that?

In any case, let's try to make the types as precise as possible. string|never is more accurate

The WP_CLI:error() here uses exit() so it indeed never, but it's returning mixed as well and not just string. So it should be mixed|never