[theme-reviewers] Required Issues Found in Live Theme

Chip Bennett chip at chipbennett.net
Sun Feb 24 13:48:11 UTC 2013

I you find issues, however you find them - diff-review, code review, unit
tests, etc. - you should note them. If the issues are at your discretion
minor enough, you can require them to be addressed in the next Theme
revision. But if they resulted from an insufficient/incomplete prior
review, you're probably better off marking them as required and resolving
as "not-approved".

(Again at your discretion you nay hold the ticket open to allow the
developer to upload a revision, then continue the review in the bew ticket.)
On Feb 24, 2013 4:23 AM, "Greg Priday" <greg at siteorigin.com> wrote:

> I just picked up this ticket http://themes.trac.wordpress.org/ticket/10913
> After giving it a full review I noticed that it's already live in the
> directory. For some reason it landed up in the "Previously Reviewed,
> Not Approved" queue.
> Should I approve the theme based on a diff review or require that the
> author fix the required issues I found?
> --
> I make free WordPress themes
> http://siteorigin.com
> http://facebook.com/SiteOrigin
> http://twitter.com/SiteOrigin
> _______________________________________________
> 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/20130224/103d2758/attachment.htm>

More information about the theme-reviewers mailing list