<div>Thanks for raising this George.</div><div><br></div><div>Both of these modules are really on fire right now (come to think of it, that's true of a lot of modules). But anyway, definitely look through them. That's what really matters early on. And it's really easy to browse the code on github [1] [2]. Hint, focus on the APIs. (Type 't' to see all files and navigate them) </div>
<meta http-equiv="content-type" content="text/html; charset=utf-8"><div><br></div><div>Unfortunately, at this point you can only comment on commits, so simply post to the mailinglist or chat in IRC if you want to provide general feedback.</div>
<div><br></div><div>On Thu, May 26, 2011 at 00:47, George Gastaldi <span dir="ltr"><<a href="mailto:gegastaldi@gmail.com">gegastaldi@gmail.com</a>></span> wrote:</div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I encourage to extend that good practice whenever is possible to other<br>
modules as well.<br>
Could be a topic to discuss on the meeting next week.<br></blockquote><div><br></div><div>George and Antoine got me thinking that perhaps we should have a "request for review" section on the project site (or some other feed mechanism) so that you can see what's currently "up for review". This should help passer-byes know when and where to jump in.</div>
<div><br></div><div>I recommend using this template for requesting a review:</div><div><br></div><div>Module: (e.g., Social)</div><div>Type: (e.g., code | documentation | example)</div><div>Focus: (e.g., APIs, type names, implementation strategy, etc)</div>
<div>Other notes: </div><div><br></div><div>The clearer the focus, the more feedback you are likely going to get (since people know what to look for).</div><div><br></div><div>-Dan</div><div><br></div></div>[1] <a href="https://github.com/seam/social">https://github.com/seam/social</a><div>
[2] <a href="https://github.com/seam/reports">https://github.com/seam/reports</a></div><div><br>-- <br><div>Dan Allen</div>Principal Software Engineer, Red Hat | Author of Seam in Action<br>Registered Linux User #231597<br>
<br><div><a href="http://www.google.com/profiles/dan.j.allen#about" target="_blank">http://www.google.com/profiles/dan.j.allen#about</a><br><a href="http://mojavelinux.com" target="_blank">http://mojavelinux.com</a><br><a href="http://mojavelinux.com/seaminaction" target="_blank">http://mojavelinux.com/seaminaction</a></div>
</div>