Use number #6. If it's not necessary, then why say it at all in a code review? Everyones got other things they'd like to do. All the "should" and "considers" a

Use number #6. If it's not necessary, then why say it at all in a code review? Everyones got other things they'd like to do. All the "should" and "considers" are tickets that can go on the backlog after a merge has delivered functionality. Then YAGNI will probably turn out to apply.