<br><br><div class="gmail_quote">On Tue, Feb 7, 2012 at 2:17 PM, Andrey Somov <span dir="ltr"><<a href="mailto:py4fun@gmail.com">py4fun@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Tue, Feb 7, 2012 at 12:26 PM, Patrick Mézard <span dir="ltr"><<a href="mailto:patrick@mezard.eu" target="_blank">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 :-)<font color="#888888"><br>

</font></blockquote></div><div><br></div><div><br></div></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></blockquote><div><br></div><div>

Or as stated by Patrick and Dirkjan:</div><div>3) Open an issue, paste the patch as a guidance for how it could be fixed. Let the community provide the patch that will be applied.</div><div><br></div><div>Cheers,</div><div>

<br></div><div>Benoit</div></div>