QA in Squeak (was: Re: [Bug][Fix] Setting copy/paste-key preference under Windowsdoesnot work)

Tim Rowledge tim at sumeru.stanford.edu
Mon Dec 6 23:15:14 UTC 2004


Avi Bryant <avi.bryant at gmail.com> wrote:

> I think the flipside of this is that the people that *have* shown
> their qualification in an area should not have to jump through the
> hoops of a review process to get their changes into (an alpha release
> of) that area.  It should be hard to become a committer, but not hard
> to commit once you are one.

I'm not convinced of that. I'd much prefer to be able to have proper
reviews by several people whenever possible. I'm trying to keep the VM
stuff 'clean' as best I can and it's somewhat helped by VMMaker being
an external package. But the practical issue is getting people's
eyeball time for reviewing; we're all pretty busy and this is entirely
volunteer so what can one do? Fire someone?

We all get to make mistakes, no matter how experienced or expert we
are. I didn't spot Andreas' mistakes in the 'isArray' VM changes
introduced in 3.6 and thus we had to bugfix later, a classic example of
trusting an experienced expert and not doing enough cross checking.
When I was at ParcPlace I got to correct a few bugs by Peter Deutsch -
Peter for ghu's sake - in the translator. 

tim
--
Tim Rowledge, tim at sumeru.stanford.edu, http://sumeru.stanford.edu/tim
Useful random insult:- One clown short of a circus.



More information about the Squeak-dev mailing list