<div dir="ltr">Greetings,<div><br></div><div>I always have bad code I've written to volunteer.  6 months ago Robert was an idiot and 6 months from now Robert is a jerk!</div><div><br></div><div>As for the actual reviews, while I like the idea of doing them "as a group" I think that may be a bit difficult to actually do.  Having a single person leading each review exposes us to their style and shows us what is important to them.  Some people might not care that you used " when no variables were being interpolated, others might have a watchful eye for mixed usage of "!" with "and".  I think by having us watch a series of live code reviews we get the most possible exposure to different styles and values.  I'm less interested in getting good code (but if anyone says I said that I'll deny it) and more interested in seeing how others actually perform the reviews and what they are looking for.</div><div><br></div><div>As an example, if John was to review I'd expect him to be looking for security issues and his bag of grep tricks, if Wade were to review I'd expect more of a focus on code smells, algorithms, and readability.  Seeing these two contrasted styles could be very interesting!</div><div><br></div><div>That being said, if you think we can make group reviews work I'm certain down to give it a shot.  Here is a list of my CPAN modules I feel might be appropriate for review:</div><div><br></div><div><b>Small, Well Isolated, Easy to Follow:</b></div><div><a href="https://metacpan.org/pod/DBIx::Class::InflateColumn::Math::Currency">https://metacpan.org/pod/DBIx::Class::InflateColumn::Math::Currency</a><br></div><div><a href="https://metacpan.org/pod/DBIx::Class::InflateColumn::Time">https://metacpan.org/pod/DBIx::Class::InflateColumn::Time</a><br></div><div><a href="https://metacpan.org/pod/Dancer::Plugin::Legacy::Routing">https://metacpan.org/pod/Dancer::Plugin::Legacy::Routing</a><br></div><div><br></div><div><b>Medium Complexity, Originally Part of a Perl Mongers Presentation:</b></div><div><a href="https://metacpan.org/pod/Game::WordBrain">https://metacpan.org/pod/Game::WordBrain</a><br></div><div><a href="https://metacpan.org/pod/Crypt::Rijndael::PP">https://metacpan.org/pod/Crypt::Rijndael::PP</a></div><div><br></div><div><b>Interesting (Command Pattern!) But Much Larger:</b></div><div><a href="https://metacpan.org/pod/WWW::eNom">https://metacpan.org/pod/WWW::eNom</a><br></div><div><a href="https://metacpan.org/pod/WWW::LogicBoxes">https://metacpan.org/pod/WWW::LogicBoxes</a></div><div><br></div><div>I offer up all of these but recommend the small and well isolated ones in the interests of time and ease of understanding.  Alternatively, the modules that resulted from presentations could be interesting to see again.</div><div><br></div><div>However we go about this though, I think it's going to be quite enlightening and fun!</div><div><br></div><div>Best Regards,</div><div>Robert Stone</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 8, 2017 at 7:21 AM, 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"><span class="">On Mon, 8 May 2017 07:11:04 -0500<br>
Julian Brown <<a href="mailto:jlbprof@gmail.com">jlbprof@gmail.com</a>> wrote:<br>
<br>
> To the first list, may we add performance improvements as well?<br>
<br>
</span>Sure.<br>
The list was meant as a starting point for discussion.<br>
<br>
G. Wade<br>
<div class="HOEnZb"><div class="h5"><br>
> Julian<br>
><br>
><br>
> On Sun, May 7, 2017 at 9:32 PM, G. Wade Johnson <<a href="mailto:gwadej@anomaly.org">gwadej@anomaly.org</a>><br>
> wrote:<br>
><br>
> > The code review session sounds like an interesting new thing to<br>
> > try. In order to make this work, we'll need to have 2-3 modules<br>
> > that people are 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<br>
> > are 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>
> ><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>
> ><br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
There are 2 possible outcomes: If the result confirms the hypothesis,<br>
then you've made a measurement. If the result is contrary to the<br>
hypothesis, then you've made a discovery.                       --<br>
Enrico Fermi<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<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>
</div></div></blockquote></div><br></div>