[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