3 ms·
> If you still teach, do you ever have your students do code review on eachother's code? Yes, I teach. No I haven't done that. But it's a great idea. I'll thin
by ggchappell 12y ago
> If you still teach, do you ever have your students do code review on eachother's code?
Yes, I teach. No I haven't done that. But it's a great idea. I'll think about it.
<ggchappell proceeds to think about it>
- civilian 12y agoIn the more hardcore-about-code-review job I had, the code reviewer was as responsible for bugs / broken stuff that came out as the person who wrote the code. And in that job we didn't just read the code briefly, you'd load up the branch and test their code. I'm not sure that it was the best practice, it slowed everything down and led to a more formal culture. If you do this, in order to incentivize the code reviewer you might actually make 10% of the grade based on the code submitted by the person they reviewed. (Granted that could be kind of harsh, and I always hated group projects in school.)
- honestcoyote 12y agoProbably best to make code review an extra credit exercise because you don't want to ever penalize a good student by tying their rewards to the actions of someone who might be slacking.