<div dir="ltr">To the first list, may we add performance improvements as well?<div><br></div><div>Julian</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, May 7, 2017 at 9:32 PM, G. Wade Johnson <span dir="ltr"><<a href="mailto:gwadej@anomaly.org" target="_blank">gwadej@anomaly.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The code review session sounds like an interesting new thing to try. In<br>
order to make this work, we'll need to have 2-3 modules that people are<br>
willing to have reviewed.<br>
<br>
Jim has volunteered some code. Anyone else?<br>
<br>
We also need to set some ground rules about the kind of review we are<br>
going for.<br>
<br>
- Level of pickiness<br>
   - Style issues?<br>
   - Bugs only<br>
   - Maintainability<br>
- Security?<br>
- Architectural improvements?<br>
<br>
I'd also suggest that people carefully think in terms of three<br>
different kinds of comments:<br>
<br>
  - Question<br>
    - Looking for clarification<br>
    - Might suggest minor change or documentation for clarity<br>
  - Comment<br>
    - Non-fatal issue that might be worth changing or considering<br>
  - Flaw<br>
    - Bug<br>
    - Logic error<br>
<br>
Obviously, we have no way of enforcing changes. And, we want to all<br>
remain friendly after the fact. We are just looking for good quality<br>
code in the end.<br>
<br>
Does this sound like an approach that everyone can agree to?<br>
<span class="HOEnZb"><font color="#888888"><br>
G. Wade<br>
--<br>
Uncertainty is an uncomfortable position. But certainty is an absurd<br>
one. -- Voltaire<br>
______________________________<wbr>_________________<br>
Houston mailing list<br>
<a href="mailto:Houston@pm.org">Houston@pm.org</a><br>
<a href="http://mail.pm.org/mailman/listinfo/houston" rel="noreferrer" target="_blank">http://mail.pm.org/mailman/<wbr>listinfo/houston</a><br>
Website: <a href="http://houston.pm.org/" rel="noreferrer" target="_blank">http://houston.pm.org/</a><br>
</font></span></blockquote></div><br></div>