[theme-reviewers] Coding Standard

[Srikanth Koneru]

can't recall any instantly, but reviewers usually point to make site and
state that its a required issue even tough its not mentioned on org
guidelines.


On Tue, Apr 30, 2013 at 2:56 AM, Chip Bennett <chip at chipbennett.net> wrote:

> What policies, specifically?
>
>
> On Mon, Apr 29, 2013 at 5:25 PM, Srikanth Koneru <tskk79 at gmail.com> wrote:
>
>> A few policies are formulated on make site but they don't make it to
>> guidelines on .org
>> so these policies are only recommended then?
>>
>>
>> On Tue, Apr 30, 2013 at 2:53 AM, 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
>>>
>>>
>>
>> _______________________________________________
>> 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/20130430/5ab6bf88/attachment.html>