[wp-trac] [WordPress Trac] #22704: Automatic Core Updates
WordPress Trac
noreply at wordpress.org
Thu Sep 19 21:42:40 UTC 2013
#22704: Automatic Core Updates
-----------------------------+-----------------------
Reporter: pento | Owner: pento
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: 3.7
Component: Upgrade/Install | Version: 3.5
Severity: normal | Resolution:
Keywords: dev-feedback |
-----------------------------+-----------------------
Comment (by DrewAPicture):
Replying to [comment:53 JustinSainton]:
> First pass at inline filter documentation. Be gentle, @DrewAPicture.
Looks pretty good for a first inline docs patch!
Just a few things:
* The short description should probably mention 'email' somewhere. I
suggest something like "Filter whether to email an update summary to the
site admin." Whatever you make it, it needs a period at the end.
* Add "Default true." to the end of the first @param line.
* The $core_update line @param is a little confusing. False on ''what''
failing? And "otherwise the core update offering" is pretty vague. So
maybe, "An array of core update data, false otherwise."
Other than that stuff, it looks great.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/22704#comment:54>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list