|
|
Microsoft (364) |
OSS (243) |
Total |
| Provide Comments |
|
296 |
208 |
504 |
|
Comments |
203 |
152 |
355 |
|
Suggestions/advice/hints/solutions |
137 |
116 |
253 |
|
Constructive criticism |
8 |
12 |
20 |
|
|
|
|
|
| Communicate with the author |
|
126 |
13 |
139 |
|
Email |
14 |
8 |
22 |
|
Voice/Face-to-Face |
119 |
6 |
125 |
|
|
|
|
| Provide Examples |
|
59 |
31 |
90 |
|
Examples- |
56 |
21 |
77 |
|
Other reviews/contributions |
6 |
13 |
19 |
|
|
|
|
| Provide Mentoring |
|
52 |
40 |
92 |
|
encourage contributions |
5 |
2 |
7 |
|
mentoring/tutoring |
7 |
16 |
23 |
|
outside resources/guides/guidelines |
34 |
26 |
60 |
|
Ask for new iteration |
4 |
0 |
4 |
|
question rationale |
8 |
0 |
8 |
|
|
|
|
| I don’t, I Reject the code |
|
40 |
27 |
67 |
|
Reject- |
1 |
10 |
11 |
|
Reject until good |
4 |
14 |
18 |
|
Give opportunity to fix and then reject |
35 |
4 |
39 |
|
|
|
|
|
| I Rewrite/fix the code |
|
10 |
48 |
58 |
|
Rewrite /fix |
10 |
48 |
58 |
|
|
|
|
|
| Other |
|
9 |
14 |
23 |
|
Other |
9 |
14 |
23 |
|
|
|
|
|