-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Ciaran McCreesh wrote:
> Oh come on, haven't you heard my rants about the state of the tree and
> the number of monkeys who have commit access?

Yes I've read those rants, among others.. :)

But with everyone screaming 'not enough manpower' the number of devs
with commit access is just bound to increase. So why not focus on how to
increase quality by default?

> Problem is, getting decent
> QA done once things hit the tree is in many cases very difficult

So why not build peer review into the process/policy? Require that the
team leads (who could deligate as they see fit) perform verification
(peer review) before closing out bugs.

> -- the
> kind of people who won't accept QA feedback are usually the kind who
> are making the worst mistakes. The maintainer-wanted list is simply an
> easier target...

True; being a premadonna isn't pretty or helpful to the project, but I
bet alot of it is due to bad expectations. There seems to be a vocal
minority of devs who equate being a dev with a God-like status. "How
dare you question me or my work?!?" And it would make sense that the new
devs would pick up on that as a way to 'fit in'. So how can we set
better expectations for new devs up front? Update the dev policy docs?:

- - Expect to have your work peer reviewed at all times
- - Realise that peer reviews are intended to improve the code not
evaluate dev performance

Ideas?

Nathan



-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)

iD8DBQFDBe572QTTR4CNEQARAtYpAJ0aZ4gnfyE4lTUrbYr/DcWmIUX67ACghyvl
TTCM9mWVTkuUWm33WnSeE9A=
=gE1q
-----END PGP SIGNATURE-----
-- 
gentoo-dev@gentoo.org mailing list

Reply via email to