[wp-trac] [WordPress Trac] #60513: trivial: using 'double asterisks (/**)' instead of 'single asterisk (/*)' for 'MockAction' class in 'tests/phpunit/includes/utils.php' looks better to me

WordPress Trac noreply at wordpress.org
Sat Feb 17 07:55:48 UTC 2024


#60513: trivial: using 'double asterisks (/**)' instead of 'single asterisk (/*)'
for 'MockAction' class in 'tests/phpunit/includes/utils.php' looks better
to me
-----------------------------------------------+---------------------------
 Reporter:  hztyfoon                           |       Owner:  audrasjb
     Type:  defect (bug)                       |      Status:  accepted
 Priority:  normal                             |   Milestone:  6.5
Component:  General                            |     Version:
 Severity:  trivial                            |  Resolution:
 Keywords:  good-first-bug commit needs-patch  |     Focuses:  docs,
                                               |  coding-standards
-----------------------------------------------+---------------------------
Changes (by rudlinkon):

 * keywords:  good-first-bug has-patch commit => good-first-bug commit
     needs-patch


Comment:

 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.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/60513#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list