On Tue, Feb 7, 2012 at 12:26 PM, Patrick Mézard <span dir="ltr"><<a href="mailto:patrick@mezard.eu">patrick@mezard.eu</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Finally, reading Idan review:<br>
<br>
  <a href="http://selenic.com/pipermail/mercurial-devel/2012-January/037663.html" target="_blank">http://selenic.com/pipermail/mercurial-devel/2012-January/037663.html</a><br>
<br>
I would say you seem to know your way in python, you write tests and apparently your fix is correct. It is all about commit message formatting and separating concerns. With little effort you can probably get your patch acked and pushed unchanged. So don't blame us trying to make you take this step and turn into a contributor :-)<br>

<span class="HOEnZb"><font color="#888888"><br>
--<br>
Patrick Mézard<br>
</font></span></blockquote></div><div><br></div><div><br></div><div>My question is answered. I now clearly see the expectations from the community. There are 2 options for a reporter:</div><div><br></div><div>1) do not send any code at all</div>
<div>2) if you send code, the code must meet all the quality criteria to be included in the code base. The reporter must have enough stamina to push the changes until they are accepted</div><div><br></div><div>Thank you.</div>
<div><br></div><div>-</div><div>Andrey</div><div><br></div><div><br></div><div><br></div><div><br></div>