1 |
-----BEGIN PGP SIGNED MESSAGE----- |
2 |
Hash: SHA1 |
3 |
|
4 |
Ciaran McCreesh wrote: |
5 |
> Oh come on, haven't you heard my rants about the state of the tree and |
6 |
> the number of monkeys who have commit access? |
7 |
|
8 |
Yes I've read those rants, among others.. :) |
9 |
|
10 |
But with everyone screaming 'not enough manpower' the number of devs |
11 |
with commit access is just bound to increase. So why not focus on how to |
12 |
increase quality by default? |
13 |
|
14 |
> Problem is, getting decent |
15 |
> QA done once things hit the tree is in many cases very difficult |
16 |
|
17 |
So why not build peer review into the process/policy? Require that the |
18 |
team leads (who could deligate as they see fit) perform verification |
19 |
(peer review) before closing out bugs. |
20 |
|
21 |
> -- the |
22 |
> kind of people who won't accept QA feedback are usually the kind who |
23 |
> are making the worst mistakes. The maintainer-wanted list is simply an |
24 |
> easier target... |
25 |
|
26 |
True; being a premadonna isn't pretty or helpful to the project, but I |
27 |
bet alot of it is due to bad expectations. There seems to be a vocal |
28 |
minority of devs who equate being a dev with a God-like status. "How |
29 |
dare you question me or my work?!?" And it would make sense that the new |
30 |
devs would pick up on that as a way to 'fit in'. So how can we set |
31 |
better expectations for new devs up front? Update the dev policy docs?: |
32 |
|
33 |
- - Expect to have your work peer reviewed at all times |
34 |
- - Realise that peer reviews are intended to improve the code not |
35 |
evaluate dev performance |
36 |
|
37 |
Ideas? |
38 |
|
39 |
Nathan |
40 |
|
41 |
|
42 |
|
43 |
-----BEGIN PGP SIGNATURE----- |
44 |
Version: GnuPG v1.4.1 (GNU/Linux) |
45 |
|
46 |
iD8DBQFDBe572QTTR4CNEQARAtYpAJ0aZ4gnfyE4lTUrbYr/DcWmIUX67ACghyvl |
47 |
TTCM9mWVTkuUWm33WnSeE9A= |
48 |
=gE1q |
49 |
-----END PGP SIGNATURE----- |
50 |
-- |
51 |
gentoo-dev@g.o mailing list |