<div dir="ltr"><div><div>Hi Antoine,<br><br></div>Thanks so much for your feedback.<br>Gemfile.lock has no new changes except for the Windows platform specific changes.I wasn't sure if that should be committed.Regarding the gherkin dependency on multi_json part,would sure look into that.<br></div><div>Thanks for reviewing the code and providing feedback.<br></div><div>Am clear with the commit message now.<br><br></div><div>Regards,<br></div><div>Jagori<br></div><div><br></div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 10, 2014 at 8:09 PM, Antoine Musso <span dir="ltr"><<a href="mailto:hashar+wmf@free.fr" target="_blank">hashar+wmf@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 10/12/2014 08:46, jagori samajdar a écrit :<br>
<span class="">> Hi,<br>
><br>
> I am confused regarding the commit messages which I should write for<br>
> each individual patch submitted against an issue.<br>
><br>
> This is in reference to the issue I had been working on:<br>
</span>> /<a href="https://gerrit.wikimedia.org/r/#/c/122400//" target="_blank">https://gerrit.wikimedia.org/r/#/c/122400//</a><br>
<span class="">><br>
> My queries are as below:<br>
><br>
</span>>  1. Should a commit message say what changes I have done in a particular<br>
<span class="">>     patch submitted against an issue or should it refer to the issue in<br>
>     general am working on?If I have reworked my code basis the inputs<br>
>     provided,then what should be my commit message? Like in my case<br>
>     :should it be "Add test for long external link" or should it be<br>
>     "Rework long external link code"[since I had reworked my code from<br>
>     the previous commit basis the inputs given]<br>
</span>>  2. How would I distinguish between individual commits?<br>
<br>
When your Gerrit change is ready and approved, only the LAST patchset is<br>
going to land in the repository. All the intermediary patchsets would<br>
only be in Gerrit and most probably never seen by other developers.<br>
<br>
If you want to comment about modifications between patchsets, use the<br>
Gerrit comment to explain reviewers the difference.  Ie the commit<br>
message would most probably stay the same between patchsets.<br>
<br>
Your commit message should reflect what the change is about, in your<br>
case:  "Add test for long external link".<br>
<br>
<br>
>  3. If I use WIP: in my commit message ,when should I remove that? If my<br>
<span class="">>     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>
</span>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>
On other repositories, I usually vote CodeReview -1 on the Change<br>
indicating it is not ready.<br>
<br>
<br>
Overall, you might want to read about the commit message guidelines<br>
recommendation:<br>
 <a href="https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines" target="_blank">https://www.mediawiki.org/wiki/Gerrit/Commit_message_guidelines</a><br>
<br>
<br>
It is a good idea to describe the problem at hand, describe your design<br>
choices and summarize the code change. Looking at your change<br>
<a href="https://gerrit.wikimedia.org/r/#/c/122400/" target="_blank">https://gerrit.wikimedia.org/r/#/c/122400/</a> , there is a few questions<br>
which might be answered in the commit message:<br>
<br>
<br>
What is this code about?<br>
Why did you need to add in Gemfile.lock the x86-mingw32 platform?<br>
Why does gherkin suddenly depends on multi_json?<br>
<br>
So you can come up with:<br>
<br>
-------------------------<br>
 [browsertest] Test for shortened external link<br>
<br>
 When a long external link is used in the VisualEditor content box, the<br>
link is shortened when shown in the external link label.<br>
<br>
  Add a new scenario to links.feature to enter a long URL and verify it<br>
is shortened.<br>
<br>
  // + Describe the Gemfile.lock changes<br>
<br>
-------------------------<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Antoine "hashar" Musso<br>
<br>
<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>
</font></span></blockquote></div><br></div>