Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor docblock improvements #5929

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Conversation

Chintesh
Copy link
Contributor

This pull request improves function comments for enhanced clarity and documentation within the codebase. The updates aim to provide clear explanations of function behaviors and parameters, improving readability and maintainability. Thank you for reviewing this contribution.

@Chintesh Chintesh requested a review from a team as a code owner April 11, 2024 18:13
@@ -506,7 +506,7 @@ public function invoke( $args, $assoc_args, $extra_args ) {
* Get an array of parameter names, by merging the command-specific and the
* global parameters.
*
* @param array $spec Optional. Specification of the current command.
* @param array $spec Optional. Specification of the current command.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks to be array{name:array<string>} as its shape. Obviously, the $spec array probably has much more, but that's the part this function needs.

The return type looks to be string[].

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @BrianHenryIE

Extra space has been removed. It is phpcs fix

@swissspidy swissspidy changed the title Enhancement : function commenting improvements Minor docblock improvements Apr 12, 2024
@swissspidy swissspidy added this to the 2.11.0 milestone Apr 12, 2024
@swissspidy swissspidy merged commit 5a91c54 into wp-cli:main Apr 12, 2024
62 of 65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants