[theme-reviewers] Commenting code out

[Edward Caissie]

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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20140731/3bcf54bc/attachment.html>