[theme-reviewers] Commenting code out

Justin Tadlock justin at justintadlock.com
Thu Jul 31 14:57:10 UTC 2014


Yeah, code commenting is not a part of our guidelines.  No need to worry
over it too much.


On Thu, Jul 31, 2014 at 7:54 AM, Edward Caissie <edward.caissie at gmail.com>
wrote:

> I don't agree with the statement as a blanket idea ... it should deal with
> the intent going forward and should be removed and/or modified to reflect
> that idea.
>
> IF the code is simply not working but is intended to be included in a
> future release, leaving it in as a noted "work in progress" item would be
> fine as far as I am concerned. This is also given that once the code is
> sorted out it will not be creating an instance where the theme should not
> be approved because of its implementation. The codex statement does not
> allow for this "style" of programming which we are not here to enforce, in
> most cases we can only recommend that best practices are followed.
>
> Edward Caissie
> aka Cais.
>
>
> On Thu, Jul 31, 2014 at 7:10 AM, Stephen Cui <scui2005 at gmail.com> wrote:
>
>> These are documentation not guidelines. I do not think the statement is
>> accurate.
>>
>> Stephen
>>
>> On Jul 31, 2014, at 6:52 AM, Thomas Ireland <info at designchapter.com>
>> wrote:
>>
>> Not allowed... Theme codex - http://codex.wordpress.org/Commenting_Code
>>
>> "Note: If you are submitting Theme to WordPress Directory, code
>> commenting should not be used."
>>
>> On 7/31/2014 11:01 AM, Michael van den Berg wrote:
>>
>> Hi everyone!
>>
>>  I'm reviewing a theme which threw a few errors. The theme author solved
>> this issue by commenting a few lines out. Is this acceptable? I guess it
>> works, but it looks a bit messy...
>>
>>  Thanks!
>>
>>  Best regards,
>>
>>  Michael
>>
>>
>> _______________________________________________
>> theme-reviewers mailing listtheme-reviewers at lists.wordpress.orghttp://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/20140731/33627dcb/attachment-0001.html>


More information about the theme-reviewers mailing list