[wp-trac] [WordPress Trac] #41057: Update PHP codebase per WordPress PHP Coding Standards

WordPress Trac noreply at wordpress.org
Thu Jun 15 12:42:02 UTC 2017


#41057: Update PHP codebase per WordPress PHP Coding Standards
----------------------------+-----------------------
 Reporter:  netweb          |       Owner:  pento
     Type:  task (blessed)  |      Status:  accepted
 Priority:  normal          |   Milestone:  4.9
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |     Focuses:
----------------------------+-----------------------

Comment (by johnjamesjacoby):

 Replying to [comment:5 pento]:
 > I think we can safely remove it.

 I agree with Gary & Mark about not continuing this one. Code editors 12
 years ago did have bracket matching, but back then it was popular to
 document the beginning & end of large HTML blocks, and this was carried
 over from that.

 As far as large coding standards related changes go:
 * We can minimize the number of times they’re necessary by enforcing the
 standards for every comimit after these go in.
 * We can evaluate the impact of new coding standards changes regularly and
 keep the codebase up to date with them.
 * We shouldn’t be afraid to to do either of those things.

 There’s nothing egregious in our current coding standards – that I can
 find – that is reason enough to block this. It’s a million times better,
 and I expect the positive response when this goes in will be huge.

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


More information about the wp-trac mailing list