<div dir="ltr"><div><div>Thanks Amir for the link.<br></div>Regards,<br></div>Jagori<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 10, 2014 at 10:25 PM, Amir E. Aharoni <span dir="ltr"><<a href="mailto:amir.aharoni@mail.huji.ac.il" target="_blank">amir.aharoni@mail.huji.ac.il</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="rtl"><span class=""><div dir="ltr">2014-12-10 11:09 GMT+02:00 Antoine Musso <<a href="mailto:hashar%2Bwmf@free.fr" target="_blank">hashar+wmf@free.fr</a>>:<br>> >  3. If I use WIP: in my commit message ,when should I remove that? If my<br>> >     work is awaiting feedback and approval from someone,should I keep<br>> >     WIP or is it if am confident enough of my code I should remove that?.<br>><br>> As far as I know, using WIP in commit message is a convention used by<br>> the VisualEditor project. I guess you can remove it once you are<br>> confident your patch can land in, but better want to check with them.<br><br></div></span><div dir="ltr">We use WIP for Language engineering projects, too.<br>For us it means "please don't merge this yet, but comments are welcome".<br>And yes, you should remove it once you think that it is ready for merging.<br></div><div dir="ltr"><br></div><div dir="ltr">Here's a blog post with useful tips for commit messages that should be generally useful for any project:<br><a href="http://infotrope.net/2013/08/24/start-your-commit-message-with-a-verb/" target="_blank">http://infotrope.net/2013/08/24/start-your-commit-message-with-a-verb/</a><br></div><div dir="ltr"><br>--<br>Amir Elisha Aharoni · אָמִיר אֱלִישָׁע אַהֲרוֹנִי<br><a href="http://aharoni.wordpress.com" target="_blank">http://aharoni.wordpress.com</a><br>‪“We're living in pieces,<br>I want to live in peace.” – T. Moore‬<br><br></div></div>
<br>_______________________________________________<br>
QA mailing list<br>
<a href="mailto:QA@lists.wikimedia.org">QA@lists.wikimedia.org</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/qa" target="_blank">https://lists.wikimedia.org/mailman/listinfo/qa</a><br>
<br></blockquote></div><br></div>