trivial: using 'double asterisks (/**)' instead of 'single asterisk (/*)' for 'MockAction' class in 'tests/phpunit/includes/utils.php' looks better to me

[hztyfoon]

Looks to me,
In here ​https://github.com/WordPress/wordpress-develop/blob/4fe09a2df/tests/phpunit/includes/utils.php#L52 for the code doc of 'MockAction' class instead of using single asterisk (/*) for the beginning of that code doc, we should use double asterisks (/**).

This will be consistent with other docs for class & also will be better for visibility/highlighting of the comment @since UT (3.7.0) in popular code editors like 'VS Code','PhpStorm' etc.

[audrasjb]

Introduced in [47122].

Thanks for spotting this small issue.
Adding good-first-bug as it seems like an ideal ticket for first time contributors :)

I changed one of the /* to a /** to follow proper PHPDoc comments syntax

Trac ticket: https://core.trac.wordpress.org/ticket/60513

[audrasjb]

Thanks for the PR @mahnewr! Self assigning for commit and moving to milestone 6.5.

[rudlinkon]

Thank you @hztyfoon for find out the issue. I think we can do the same fix for some places also. I have given two more places where we can apply the same fix.

​https://github.com/WordPress/wordpress-develop/blob/4fe09a2df3a3cd8360228051955f66ebf1033374/tests/phpunit/includes/bootstrap.php#L59

​https://github.com/WordPress/wordpress-develop/blob/4fe09a2df3a3cd8360228051955f66ebf1033374/tests/phpunit/includes/wp-profiler.php#L3

Also thanks for the PR @mahnewr I requested you to do the same fix for those places.

@mahnewr thanks for the PR, I think we can do the same fix for some places also. I have given two more places where we can apply the same fix.

​https://github.com/WordPress/wordpress-develop/blob/4fe09a2df3a3cd8360228051955f66ebf1033374/tests/phpunit/includes/bootstrap.php#L59
​https://github.com/WordPress/wordpress-develop/blob/4fe09a2df3a3cd8360228051955f66ebf1033374/tests/phpunit/includes/wp-profiler.php#L3

I would not change the bootstrap.php since that appears to be a simple multi line comment (see 5.2 of ​https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/ )

The WP Profiler looks like it could function as the docblock for the class, so it should and have the additional check for completeness, like @ since, etc.

Ready for commit 🎉

I would not change the bootstrap.php since that appears to be a simple multi line comment (see 5.2 of ​https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/ )

The WP Profiler looks like it could function as the docblock for the class, so it should and have the additional check for completeness, like @ since, etc.

Thank you @kraftbj so we can add double asterisks on the wp-profiler.php file.

This is an additional change of ​PR#6132

Trac ticket: https://core.trac.wordpress.org/ticket/60513

[swissspidy]

In 57665:

Build/Test Tools: Fix docblocks in a couple of test files to use two asterisks.

Props mahnewr, rudlinkon,kraftbj, audrasjb, mukesh27.
Fixes #60513.

Fixed in https://core.trac.wordpress.org/changeset/57665

Fixed in https://core.trac.wordpress.org/changeset/57665