wiki:org/meetings/2015SummerDevMeeting/CodeReviews

Doing a Code Review

What is the purpose of a code review?

Get the code

  • Get branch
  • Read diffs
  • Compile (without warnings)
  • make check (all success, no warnings in log)
  • make test-network-all (requires chutney, all success)
  • make check-spaces

Structure

Style

Security

Additional Tools

  • static analysis
  • dynamic sanitisers:
    • address
    • undefined behaviour
    • memory
Last modified 2 years ago Last modified on Oct 2, 2015, 4:12:02 PM