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]

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

[Copilot]

Pull request overview

This PR corrects inline documentation for the return_value() method in the Config_Command class. The changes fix a duplicate parameter name and update the type annotations.

Changes:

• Fixed duplicate @param entry for $type - replaced with correct $values parameter
• Updated @param type annotation for $values to reflect its array structure
• Updated @return annotation to describe the return type

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!