- User Since
- Jun 16 2015, 9:28 PM (251 w, 2 d)
Jun 17 2015
Also, feel free to re-close if they are actually the same issue (sorry :/)
Sorry but I'm not entirely sure that these are actually the same issue:
In t5815, btrahan specifically said that he was not going to implement a "quick accept" button. Instead was planning to implement a "go to next" mode.
First, thanks so much for adding 'done' comments, it's definitely helps. I can speak to why we asked for this a little bit.
So, here's one example:
We have this file "etherpad/src/notes/control/maincontrol.ts". Right now the "history" page shows:
Jun 16 2015
Thanks for the response. There is absolutely review going on. Until my team launches our product we will depend pretty heavily on post-commit reviews, so it's not uncommon for me to have 50-150 audits in one week to review. A lot of those will be really small commits, only a couple of lines, so I can accept them pretty quickly. Probably 90% of the audits will be accepted without any comments right now, so I guess this is a request to maybe try to optimize for that?