The focus should be on the attributes listed in the Reviewing charms post, specifically in the review criteria section (which is essentially the same as the corresponding section in the spec).
I think that we do have an issue at the moment that sometimes there is more of a code review than focusing on the criteria we have defined for listing (maybe a code review should be added, and certainly it can have valid as you imply, but it’s indeed not really in the current list, other than “coding conventions”. I had a stretch item on this cycle’s roadmap to try to make the review process clearer and smoother, but it’s looking unlikely that I’ll be able to get to that before The Hague - I’ll try to make sure that Charm-Tech gets to it next cycle.
@arturo-seijas: as @gruyaume said, the code review is helpful, but could you please focus on the review criteria? The easiest way to do this is to copy the checklist into a comment here and tick off (or not!) each item as you verify it. Thanks!