Ian Clatworthy - is he coreworthy or not?

[Ian Clatworthy]

John Arbash Meinel wrote:

> I'm of 2 minds:
> 
> 1) We gave him the Release Manager role for 0.17. And anyone with RM should
> have pqm access.

I could be wrong here but it looks like I need it in order to do the RM
things expected: http://bazaar-vcs.org/ReleaseChecklist. I'm in no
particular hurry otherwise.

> 2) He is doing good work, but has not done so through a complete release.

Agreed.

> (1) is sufficient for me to give him "core" status. He has spent a lot of time
> to focus on the code (printing out pages and pages of hardcopy to read???).
> I also trust that he won't +1 things unless he actually feels he understands
> the code in question.

On the change front, I've mostly tried to stick to lower risk areas
(doc, tests, etc.) until I'm comfortable with more of the code and how
the team likes things done.

On the review front, I've equally tried to be very explicit about the
scope of my feedback, e.g. the partial review of Aaron's faster
TreeTransform work.

If I ever start approving things I don't understand, feel free to tell
me *loudly* that I don't know what I don't know! If the last few months
are any guide, you'll undoubtedly do that more politely than I would
together with a nicely written, very clear explanation of things. :-)

BTW, I'm pleased I didn't have to go through school with "Codeworthy" as
a last name - Clatworthy was bad enough!

Ian C.