Add reviewing checklist to docs/contributions.

There is some overlap with the style guide. Where the style guide aims to be descriptive, this document is simply a list of questions for the reviewer to ask when when reading a PR.
specifically, do tearDown{Test,Suite} assume that their companion setUp{Test,Suite} functions completed successfully ?