Functional : Does it work?
Are the requirements clear from
Does the code execute as expected? (Does is meet the requirement of the issue/task)
Are resources, events and memory released in all paths? (Disposable called?)
...
Documentation Is it (self)documented?
Are all functions, methods and classes documented? (Summary,parameters,comments etc)
Future questions
Does the code generate warnings? (this question is not doable now because we have 800+ warnings)
Conventions
DS Coding conventions are here : http://public.deltares.nl/display/DS10/Development+Conventions+in+Delta+Shell
And general conventions here : http://msdn.microsoft.com/en-us/library/ms229042.aspx
There are also resharper 4.5 setting here resharper.xml (Resharper->Options->Common->Code Styling Sharing->Import)