<div dir="ltr">Thank you James for the invitation. We are more than happy to join the organisation and help communtiy in reviewing and merging pull requests. :)<div><br></div><div>Thank you for providing such detailed information, I have been through all the links you have provided. I understand the role of a reviewer much better now.</div><div><br></div><div>><span style="font-size:12.799999807338525px"><font color="#783f04">Should the above be added to sugar-docs?</font></span></div><div><font color="#000000"><span style="font-size:12.799999807338525px">Very well written guidelines for any new member to the organisation. +1 for sugar-docs from me.</span></font></div><div><br></div><div><font color="#000000"><span style="font-size:12.799999807338525px">Now that my exams are over, I look forward to a more active participation. cheers!</span></font></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 23, 2018 at 3:25 AM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Rahul and Yash, your code contributions have been consistently good<br>
for the past month, so I've invited you to the GitHub sugarlabs<br>
organisation so that you can review and merge pull requests.<br>
<br>
Information below may be of help to guide you in this task.<br>
<br>
<br>
My goals for review are;<br>
<br>
- detect trivial mistakes,<br>
<br>
- maintain consistent and good code quality,<br>
<br>
- reproduce test results, (especially for critical repositories),<br>
<br>
- maintain a useful git commit history for use by git bisect, and<br>
  developers who read it,<br>
<br>
- maintain other records, such as issues, tickets, and documentation,<br>
<br>
- not waste the time of the contributor, by doing myself anything<br>
  trivial that otherwise the contributor might have to do.<br>
<br>
<br>
Checklist for review of pull requests;<br>
<br>
- [ ] does the change have consensus of the community,<br>
      <a href="https://github.com/sugarlabs/sugar-docs/blob/master/src/CODE_OF_CONDUCT.md" rel="noreferrer" target="_blank">https://github.com/sugarlabs/s<wbr>ugar-docs/blob/master/src/CODE<wbr>_OF_CONDUCT.md</a><br>
      (if a reviewer is in doubt, seek opinions by @mentioning people)<br>
<br>
- [ ] does the commit message explain the summary, problem, and<br>
      solution, so that it can be used in future analysis,<br>
      <a href="https://github.com/sugarlabs/sugar-docs/blob/master/src/contributing.md#making-commits" rel="noreferrer" target="_blank">https://github.com/sugarlabs/s<wbr>ugar-docs/blob/master/src/cont<wbr>ributing.md#making-commits</a><br>
      (if a reviewer can fix it by squash or manual rebase, do so)<br>
<br>
- [ ] does the commit message reference the issue, <a href="http://bugs.sugarlabs.org" rel="noreferrer" target="_blank">bugs.sugarlabs.org</a><br>
      ticket number, or downstream ticket numbers,<br>
      (if a reviewer can fix it by squash or manual rebase, do so)<br>
<br>
- [ ] are the number of commits excessive for future analysis,<br>
      (a reviewer may squash or rebase if necessary)<br>
<br>
- [ ] is the changed code consistent in style with the existing code,<br>
      <a href="https://github.com/sugarlabs/sugar-docs/blob/master/src/desktop-activity.md#coding-standards" rel="noreferrer" target="_blank">https://github.com/sugarlabs/s<wbr>ugar-docs/blob/master/src/desk<wbr>top-activity.md#coding-standar<wbr>ds</a><br>
      (on the other hand, expect flake8 changes to be in separate commits)<br>
<br>
- [ ] for critical repositories, does the change work properly on our<br>
      latest version of Sugar on either Fedora, Debian, or Ubuntu.<br>
<br>
<br>
Critical repositories are;<br>
<br>
- sugar, sugar-toolkit, sugar-toolkit-gtk3, sugar-artwork,<br>
  sugar-datastore, gst-plugins-espeak,<br>
<br>
- each of the Fructose activity set repositories,<br>
  <a href="https://wiki.sugarlabs.org/go/Development_Team/Release/Modules#Fructose" rel="noreferrer" target="_blank">https://wiki.sugarlabs.org/go/<wbr>Development_Team/Release/Modul<wbr>es#Fructose</a><br>
<br>
<br>
Comments?  Should the above be added to sugar-docs?<br>
<span class="m_2414849799690734410HOEnZb"><font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.netrek.org/" rel="noreferrer" target="_blank">http://quozl.netrek.org/</a><br>
</font></span></blockquote></div><br></div></div>