<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2018-03-08 17:03 GMT+01:00 Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Hi Alistair,<br>
<span class=""><br>
> On Mar 8, 2018, at 7:34 AM, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:<br>
><br>
> Hi Sean,<br>
><br>
><br>
>> On Mar 8, 2018, at 6:06 AM, Sean P. DeNigris <<a href="mailto:sean@clipperadams.com">sean@clipperadams.com</a>> wrote:<br>
>><br>
>><br>
>> Hi guys, forgive my potentially naive intrusion, but I've noticed a lot of<br>
>> git commits which seem like noise. When I make commits like these - e.g.<br>
>> tweaking tiny details to pacify CI - I call it "programming by flopping<br>
>> around" ha ha. Anyway, as I've recently learned, a great practice is to<br>
>> perform these on another branch until things are working, and then squash<br>
>> all the commits into one before merging - usually pretty easy with git. That<br>
>> having been said, I know very little about the VM dev process, so if none of<br>
>> this applies, feel free to consider me a crazy old man mumbling things on<br>
>> the street corner to passersby ;)<br>
><br>
> Those commits were on a branch.  I agree that it's best not to push after each commit.  The emails however confused me too.  Is it possible for the commit emails to clearly state the branch name?<br>
<br>
</span>Sorry.  I meant yo say the title of the emails.  In Tobias' defence, if one is working on the CI then, without a lot of effort, one has no choice other than to commit so that the CI can pull and do its thang.  Perhaps turning off the notifications for certain branches is possible?<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div>
<div><br>Personnally, i prefer to see the individual commits rather than a big ball of changes.<br></div><div>Even the mistakes are valuable, we may learn something.<br></div><div>I'm not a fan of squashing nor any other attempt to rewrite history.<br></div><div><br></div><div>I've been away from vm dev for a few months and was also a bit confused by proliferation of commit mails, BUT:<br><br>
<div>1) as Alistair noticed, the branch name is explicitely written in the mail.<br></div><div>So I should learn to better decipher those messages<br></div><div><br></div><div></div>

</div><div>
<div></div>2) I'm unsure about the benefits of turning the notifications off.<br></div><div>It's always possible to fork the repository and work in a clone, the clone can benefit from travis, and the mailing list won't be notified by the clone.<br><br></div><div>Branching directly in a common repository has some benefits:<br></div><div>- the work is publicized<br></div><div>- the work is more easily reviewable<br></div><div>- team can more easily collaborate (commit) on the branch<br></div><div> (easier than going thru PR in forks)<br><br></div><div>Nicolas<br></div><div><br></div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
>><br>
>><br>
>><br>
>> -----<br>
>> Cheers,<br>
>> Sean<br>
>> --<br>
>> Sent from: <a href="http://forum.world.st/Squeak-VM-f104410.html" rel="noreferrer" target="_blank">http://forum.world.st/Squeak-<wbr>VM-f104410.html</a><br>
</div></div></blockquote></div><br></div></div>