Proposal: Require reviewer to smoke test before merge

[akhenry]

Summary

Reviewers should test changes by running through provided test procedures once before merge.

Justification

• Ensures that test procedures exist
• Ensures that test procedures are sufficient for a third party to test the change.
• Finds issues that will block testing before the change even goes in.
• All of the above will save significant time in testathons.

Note

This means that neither the author nor the reviewer should test a change in testathon. As well as more efficient testathons, we get a third set of eyes on every change.