Monday, October 31, 2022

Code Review

Disclaimer: this is just a short recap of what Martin Dráb explained in his article https://dev.goshoom.net/2012/08/purpose-of-code-reviews/.

Purpose of code reviews

  • Internal quality:
    1. Overall approach – does the code do the right thing by proper means? 
    2. Maintainability/extensibility – are we able to easily extend the solution, fix bugs etc.? 
    3. Testability – are we able to test the code?
    4. Documentation – are we able to understand why the code was made and what it does?
    5. Performance – is the design correct from performance perspective? 
  • Functional testing: Some bugs are much more visible when looking into code
  • Knowledge sharing: Discussions between the developer and the reviewer

Tools

  • BP check 
  • xppbp.exe
  • DevOps code review request

Top 10 issues discovered from Dynamics AX Code Review

https://www.linkedin.com/pulse/top-10-issues-discovered-from-dynamics-ax-code-review-caillet/

No comments: