Skip to content

Conversation

@petitphp
Copy link
Contributor

@petitphp petitphp commented Sep 7, 2023

No description provided.

@petitphp petitphp requested a review from a team as a code owner September 7, 2023 08:19
* @param mixed $data The data to read/manipulate by reference.
* @param string|int $key The key/property the data belongs to.
* @param static $parent
* @param static $parent_instance
Copy link
Member

Choose a reason for hiding this comment

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

Let's fix the docblock fully while we're at it.

Suggested change
* @param static $parent_instance
* @param static|null $parent_instance The parent instance of the traverser.

Also, could you adapt the alignment in the docblock? I cannot do that properly via a suggestion...

Copy link
Member

Choose a reason for hiding this comment

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

Updated in 1c2e9fe

@danielbachhuber danielbachhuber added the scope:documentation Related to documentation label Sep 8, 2023
@danielbachhuber danielbachhuber added this to the 2.5.4 milestone Sep 8, 2023
@danielbachhuber danielbachhuber self-requested a review September 8, 2023 13:01
@danielbachhuber danielbachhuber merged commit 103c335 into wp-cli:main Sep 8, 2023
@petitphp petitphp deleted the fix/phpdoc-typo branch September 8, 2023 13:36
@petitphp
Copy link
Contributor Author

petitphp commented Sep 8, 2023

Thanks @danielbachhuber !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:documentation Related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants