Vault7: CIA Hacking Tools Revealed
Navigation: » Latest version
Owner: User #524297
Code Review
THOU MUST
- read code at your workstation
- use provided tools (Stash pull-request)
-
If your code is being reviewed: Address all provided comments.
- it takes a lot of time to read code. you should respond to all comments.
- If you are reviewing code: Take the time to understand it fully.
THOU SHOULD
- build and run
- provide constructive comments
- review in small logical chunks of code (a source file or module)
THOU SHOULD NOT
- Skim code superficially and blindly click 'Accept'
- Read code on a projector in a conference room (zzzzzz...)
-
Make code style comments just to start a flame war
THOU MUST NOT
-
Conduct a Code Review after a project has already been delivered to IV&V. (What's the point? You can't incorporate any recommendations, and you'll bore everyone walking through the entire codebase!)