<div dir="ltr">Hi all,<div><br></div><div>I'd like to say that that I too am thankful for all the work previously done so far. Reviewing someone else's code in detail is not always the most pleasant of past-times; hopefully we can continue to have good conversations for all future pull requests.</div><div><br></div><div>Speaking of pull requests, it may be worth prodding those open pull requests with input waiting on the submitter to see if it's something they really intend to finish and/or resolve.. since I notice that even the most recent pull requests mostly seem to have been last commented/updated 10 to 20 days ago. Maybe by those already involved in each pull request's discussion; so the pulls can be either finalised or abandoned. Oh, and I'll like to plug for help again for my Sherlock music player pull, which is still waiting on anyone familiar with ScummVM's MidiPlayer class to help me get it working :)</div><div><br></div><div>Regards,</div><div><br></div><div>Paul.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 14, 2016 at 12:46 PM, Peter Bozsó <span dir="ltr"><<a href="mailto:bozpet92@gmail.com" target="_blank">bozpet92@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 dir="ltr">Hi all!<div><br></div><div>I can't agree more with Arnaud and I am also thankful for all the work you guys have done so far.</div><div><br></div><div>All the best,</div><div>Peter</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 14 July 2016 at 15:32, Arnaud Boutonné <span dir="ltr"><<a href="mailto:strangerke@scummvm.org" target="_blank">strangerke@scummvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Eugene</div><div><br></div><div>Thanks for this detailed mail about the changes, it's clarifying a lot of things.</div><div>I hope we'll still have some good code reviews as we had until now :)</div><div><br></div><div>I'm impatient to see the next engine PR to see how it's applied!</div><div><br></div><div>Thanks to you and wjp (and LordHoto, but he won't read this mail I guess) for your work on the Core Team</div><div><br></div><div>Best regards,</div><div>Arnaud</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Thu, Jul 14, 2016 at 11:31 AM, Eugene Sandulenko <span dir="ltr"><<a href="mailto:sev@scummvm.org" target="_blank">sev@scummvm.org</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Hi Team,<div><br></div><div>I had a nice chat with wjp over a beer, and we were discussing how we may proceed with ScummVM project since LordHoto left.</div><div><br></div><div>There are going to be several separate e-mails on the process, so we have enough room for discussion. I am looking for your thoughts and obviously seeking support.</div><div><br></div><div>So, after almost 5 years of existence of the Core Team, I am declaring it as officially disbanded.</div><div><br></div><div>In the past, we had the Core Team involved mostly in the OSystem development and architecture, and now that role comes back to the individual developers.</div><div><br></div><div>That means that there is also an adjustment to the PR approval. Now the (slightly modified) procedure will be as follows:</div><div><ol><li>For a sizeable, significant changes it is advised to make a Pull Request on the GitHub<br></li><li>Such Pull Request will have to stay for at least 2 weeks open for the comments<br></li><li>Everyone is invited to comment and review and voice their opinion (views of non-team members are valuable, but have no decisive power)<br></li><li>If there are no unaddressed objections after 2 weeks, the PR could be merged. Exception could be made if there are suggestions over refactoring or tidying up the code, granted that they will be addressed in-tree</li><li>Immediately after the merge the PR maker ensures that the buildbot stays happy and is not worsened (historically we had few ports broken for months).</li></ol></div><div>Now on the sizeable changes. First of all, this is up to the discretion of the PR creator, but normally I expect that anything which breaks existing the OSystem API and the Common code, or significantly extends these, especially if it requires more work from the Porters, should go via a Pull Request process. The goal is to ensure that everything stays maintainable.</div><div><br></div><div>Any developer is free to open the PRs for less significant changes, and that process could be used very well for facilitating discussion and collecting feedback, but when there are no big changes involved, no 2 weeks timeout is enforced.</div><div><br></div><div>Individual engine sub-teams decide by themselves on the process. As I am aware, currently the SCI development is performed via PRs as well for any sizeable changes. It is up to that sub-team to drop this rule any time or make something even stricter. Just make sure you continue enjoying hacking on the ScummVM.</div><div><br></div><div>That sums up this change on the Core Team.</div><span><font color="#888888"><div><br></div><div><br></div><div>Eugene Sandulenko</div><div>ScummVM Team Lead</div></font></span></div>
<br></div></div>------------------------------------------------------------------------------<br>
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic<br>
patterns at an interface-level. Reveals which users, apps, and protocols are<br>
consuming the most bandwidth. Provides multi-vendor support for NetFlow,<br>
J-Flow, sFlow and other flows. Make informed decisions using capacity planning<br>
reports.<a href="http://sdm.link/zohodev2dev" rel="noreferrer" target="_blank">http://sdm.link/zohodev2dev</a><br>_______________________________________________<br>
Scummvm-devel mailing list<br>
<a href="mailto:Scummvm-devel@lists.sourceforge.net" target="_blank">Scummvm-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel" rel="noreferrer" target="_blank">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>
<br></blockquote></div><br></div>
<br>------------------------------------------------------------------------------<br>
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic<br>
patterns at an interface-level. Reveals which users, apps, and protocols are<br>
consuming the most bandwidth. Provides multi-vendor support for NetFlow,<br>
J-Flow, sFlow and other flows. Make informed decisions using capacity planning<br>
reports.<a href="http://sdm.link/zohodev2dev" rel="noreferrer" target="_blank">http://sdm.link/zohodev2dev</a><br>_______________________________________________<br>
Scummvm-devel mailing list<br>
<a href="mailto:Scummvm-devel@lists.sourceforge.net" target="_blank">Scummvm-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel" rel="noreferrer" target="_blank">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>
<br></blockquote></div><br></div>
</div></div><br>------------------------------------------------------------------------------<br>
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic<br>
patterns at an interface-level. Reveals which users, apps, and protocols are<br>
consuming the most bandwidth. Provides multi-vendor support for NetFlow,<br>
J-Flow, sFlow and other flows. Make informed decisions using capacity planning<br>
reports.<a href="http://sdm.link/zohodev2dev" rel="noreferrer" target="_blank">http://sdm.link/zohodev2dev</a><br>_______________________________________________<br>
Scummvm-devel mailing list<br>
<a href="mailto:Scummvm-devel@lists.sourceforge.net">Scummvm-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel" rel="noreferrer" target="_blank">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>
<br></blockquote></div><br></div>