Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

None of them engage with the content of the article either - having a "play next" button for code review is awesome assuming it works reasonably well. I'm curious about the quality of nudgebot reviews. In my experience the PRs that sit around forever are the 3000 line epic find/replace refactors all done in one commit that are impossible to really review. I skimmed the paper and didn't see any accounting for "diff time to diff length", so I'm not sure the result there is anything meaningful. Maybe people are getting faster feedback to not submit such shitty PRs.


what stops you from reviewing the sed one-liner that created the diff, instead?


Reviewing the sed isn't going to help you spot the instance that shouldn't have been replaced or the missing substitution.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: