Embed Notice

@deepbluev7 @charliermarsh

If a code review were being done as a *discussion between peers*, then it would be far more appropriate to say, "Hmmm… I was thinking I would have done it this other way. But are there advantages to this way?" And that opens up a discussion. A text comment, marked as a required "todo," blocking merge, is a challenge, not a discussion.

During discussion, the reviewer might agree that what the submitter did was the best choice.