[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