<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 20 July 2015 at 15:33, Ian Cordasco <span dir="ltr"><<a href="mailto:ian.cordasco@rackspace.com" target="_blank">ian.cordasco@rackspace.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><br>
</div></div>Yes, Matt should be doing more reviews. The difference is that (as I<br>
understand it) we want to include Matt as a documentation core more than<br>
as a core reviewer for playbooks. Matt's reviews have all pertained to<br>
documentation. Other projects (e.g., Neutron and maybe Glance) are trying<br>
out a trust system where cores are accepted for subsystem expertise. Those<br>
cores are expected to only +2 and/or approve changes for their subsystem.<br>
I don't think anyone can question Matt's expertise in documentation, or<br>
Matt's knowledge of the architecture of the project.<br>
<br>
Tl;dr What if we trust Matt to only approve documentation related changes?<br></blockquote><div><br></div><div>At this point we have very few documentation changes. I would prefer to see Matt challenging us to do better/more in that regard. He doesn't need to be a core to do that - he just needs to review the code more and share his thoughts on where documentation should be changed or added as part of the review so that docs changes form part of review patch sets.</div></div>
</div></div>