[wp-trac] [WordPress Trac] #26185: PHPDoc updates for /wp-includes/functions.php

WordPress Trac noreply at wordpress.org
Fri Apr 25 21:09:27 UTC 2014


#26185: PHPDoc updates for /wp-includes/functions.php
----------------------------+-----------------------------
 Reporter:  morganestes     |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  normal          |   Milestone:  Future Release
Component:  Bootstrap/Load  |     Version:  3.7
 Severity:  normal          |  Resolution:
 Keywords:  needs-patch     |     Focuses:  docs
----------------------------+-----------------------------
Changes (by DrewAPicture):

 * milestone:  Awaiting Review => Future Release


Comment:

 Replying to [comment:8 morganestes]:
 > I'm going back through this after 3.9 and will try again. This time, I'm
 adding a diff per function with major changes, and a combined one for just
 whitespace changes.

 Great. Honestly, I'd prefer you group up PHPDoc changes for multiple
 functions in patches. You don't need to do the whole file in one patch,
 but I'd rather not have 40 tiny patches either.

 Also, you can skip the whitespace changes. The only time whitespace
 changes are really accepted is when they're on lines already affect by
 code-level changes. This is to prevent unnecessary churn in the codebase.

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


More information about the wp-trac mailing list