[theme-reviewers] Coding Standard

Stephen Cui scui2005 at gmail.com
Mon Apr 29 21:28:35 UTC 2013


Thanks Chip and Emil. I will make the changes for my theme. But I will NOT
enforce when I review other themes.



Zhijian (Stephen) Cui
Cell: (609)977-5089
eMail: scui2005 at gmail.com


On Mon, Apr 29, 2013 at 4:23 PM, Chip Bennett <chip at chipbennett.net> wrote:

> Yes. At the moment, the Coding and CSS standards are only *recommended*.
> Making them *required* may be a bridge we cross at some point, but we're
> not there right now.
>
>
> On Mon, Apr 29, 2013 at 5:07 PM, Emil Uzelac <emil at uzelac.me> wrote:
>
>> If this does not cause any issues, I would say recommended.
>>
>> Emil
>>
>>
>> On Mon, Apr 29, 2013 at 3:53 PM, Stephen Cui <scui2005 at gmail.com> wrote:
>>
>>> Hi,
>>>
>>> During the theme review, my theme has a "required issue" for next
>>> version:
>>>
>>> There are inconsistency in spacing around parenthesis throughout the
>>> theme. Recommend to read the WordPress? PHP coding standards.
>>>
>>> http://make.wordpress.org/core/handbook/coding-standards/php/#space-usage
>>>
>>> I have no problem to make such changes as it is good practice. I also
>>> take the chance to revisit the code.
>>>
>>> My question is " is it required?" or "just recommended".
>>>
>>> Regards
>>>
>>> Stephen
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130429/8a6b025b/attachment.html>


More information about the theme-reviewers mailing list