[wp-trac] [WordPress Trac] #28829: Updating admin-bar.php for coding standards

WordPress Trac noreply at wordpress.org
Fri Jul 11 17:49:59 UTC 2014


#28829: Updating admin-bar.php for coding standards
-------------------------+-----------------------------
 Reporter:  welcher      |       Owner:
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:
Component:  Toolbar      |     Version:  trunk
 Severity:  normal       |  Resolution:  wontfix
 Keywords:               |     Focuses:  administration
-------------------------+-----------------------------

Comment (by DrewAPicture):

 Replying to [comment:2 welcher]:
 > Replying to [comment:1 SergeyBiryukov]:
 > > Please note that formatting cleanup tickets are generally discouraged.
 Coding style fixes should be a result of more substantial changes:
 http://wpdevel.wordpress.com/2011/03/23/code-refactoring/. But thanks for
 the patch!
 >
 > Thanks for letting me know - I'll do some actual work next time :)

 It's definitely "actual work" to make updates like this, so thank you for
 the patch just the same. The reasoning behind only updating for coding
 standards when the lines would be changing anyway is to reduce undue churn
 in the codebase, that's all :)

 If you're interested in contributing in ways similar to this, the docs
 team does accept patches that fix inline documentation in WordPress core.
 See #25412 for a recent example of this.

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


More information about the wp-trac mailing list