[pm-h] May Houston.pm meeting
Julian Brown
jlbprof at gmail.com
Mon May 8 05:11:04 PDT 2017
To the first list, may we add performance improvements as well?
Julian
On Sun, May 7, 2017 at 9:32 PM, G. Wade Johnson <gwadej at anomaly.org> wrote:
> The code review session sounds like an interesting new thing to try. In
> order to make this work, we'll need to have 2-3 modules that people are
> willing to have reviewed.
>
> Jim has volunteered some code. Anyone else?
>
> We also need to set some ground rules about the kind of review we are
> going for.
>
> - Level of pickiness
> - Style issues?
> - Bugs only
> - Maintainability
> - Security?
> - Architectural improvements?
>
> I'd also suggest that people carefully think in terms of three
> different kinds of comments:
>
> - Question
> - Looking for clarification
> - Might suggest minor change or documentation for clarity
> - Comment
> - Non-fatal issue that might be worth changing or considering
> - Flaw
> - Bug
> - Logic error
>
> Obviously, we have no way of enforcing changes. And, we want to all
> remain friendly after the fact. We are just looking for good quality
> code in the end.
>
> Does this sound like an approach that everyone can agree to?
>
> G. Wade
> --
> Uncertainty is an uncomfortable position. But certainty is an absurd
> one. -- Voltaire
> _______________________________________________
> Houston mailing list
> Houston at pm.org
> http://mail.pm.org/mailman/listinfo/houston
> Website: http://houston.pm.org/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.pm.org/pipermail/houston/attachments/20170508/2ce307c0/attachment.html>
More information about the Houston
mailing list