7

This is how my coworker does code reviews... imagine comment on a line of code:

"Oh why does this not do this and that.
Edit: Oh I've read few lines below, now I understand. But this may break when this and that.
Edit: Nevermind I realized, that never happens, forget about it."

Let me just emphasise, that this is single comment, so it's basically editted multiple times, before submition.

Like wtf, just delete the comment then?!

Comments
  • 3
    But if there aren't 30 (unneeded) comments it will look like I didn't review it properly!
  • 2
    @SevenDeadlyBugs I'm fine with reviewer leaving a question, or some "just in case" remark. But isn't having multiple edits just lazy? What if everyone on devrant would write like that (:
Add Comment