<div dir="ltr">Hi Christoph,<div><br></div><div>Thanks for the submissions.  One consistent pattern I've noticed about this community is that it tends to slow down during the northern-hemisphere summer.  As far as process, you got it exactly right.  A trunk committer will review them and either commit them to trunk or propose a change.  After a few iterations of this, board will vote you to have trunk access directly, but it's still recommended to start out enhancements in the Inbox first.</div><div><br>In addition to reviewing code, what I generally like to do is put "time" to work _for_ me, by loading people's Inbox submissions into my image and just running and doing development with them for a while.  This tends to exercise the usability and also expose possible corner cases.  It also ensures they won't get "forgotten" before the next Squeak release, because they're marked a dirty package in the Monticello browser.</div><div><div><br></div><div>I'm sure someone will review them pretty soon.  Thanks and welcome to the community.</div><div><br></div><div> - Chris<br><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 29, 2019 at 1:00 PM Christoph Thiede <<a href="mailto:christoph.thiede@student.hpi.de" target="_blank">christoph.thiede@student.hpi.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all,<br>
<br>
over the past weeks, I uploaded several proposals into the inbox. There are<br>
a few commits that haven't been resolved yet:<br>
<br>
- Implement SyntaxErrorNotification>>#resume: (Compiler-ct.405 and<br>
Tests-ct.412)<br>
- Fix SyntaxErrorNotification>>#messageText (Compiler-ct.406)<br>
- Fix two Compiler bugs (Compiler-ct.404 and Compiler-ct.407)<br>
- SyntaxError: Preselect error message, and enable styling (Morphic-ct.1492<br>
and Tools-ct.851)<br>
- Implement #spawn on SearchBar (Tools-ct.850)<br>
- Integrate TestRunner into Browser (SUnitTools-ct.5 and SUnitGUI-ct.70)<br>
- Introduce context-dependent styling for Shout (ShoutCore-ct.66)<br>
- Add Model>>#aboutToStyle:forMorph (ToolBuilder-Morphic-ct.228)<br>
- Allow styling in Inspector's and Debugger's value panes (Tools-ct.852,<br>
Tools-ct.853, and Tools-ct.854)<br>
<br>
I would be very interested to know what is happening with them: Will anyone<br>
review these changes, do you think they should be reworked? What are the<br>
prerequisites for a commit to be released into Trunk? I am new to the<br>
mailing list and not yet familiar with the usual processes. Should I always<br>
write a separate email to ask for reviews?<br>
I would be very happy about some feedback to my proposals! :)<br>
<br>
Best,<br>
Christoph<br>
<br>
<br>
<br>
--<br>
Sent from: <a href="http://forum.world.st/Squeak-Dev-f45488.html" rel="noreferrer" target="_blank">http://forum.world.st/Squeak-Dev-f45488.html</a><br>
<br>
</blockquote></div>