Call
To
Arms!

QL

Engineering Review Processes

 

 

  • Story Review
  • Design Review
  • Code Review

 

 

Story Review

Business Analyst

  • Schedule the meeting
  • Review the user story.
  • Present several alternatives.

BA & Engineer

  • Analyze and understand each solution's implications.
  • Choose the BEST one.

Design Review

Assigned Engineer 

  • Examines source code.
  • Produces Engineering Document.​
  • Schedules the meeting.

 

Engineering Document

  • User story / Task summary.
  • Proposed changes.
  • Affected files and systems.​
  • Unit testing plan.

Design Reviewer

  • Project standards.
  • Mise-en-place.
  • Avoid duplication.
  • Marks review task as done.

BA & QA

Clarify and Familiarize

Code Review

Assigned Engineer

  • Review file & system changes.
  • Show unit tests.

Code Reviewer 

  • Run/Check CI, code sniff and unit tests.
  • Ask questions about implementation.
  • Correct any basic mistakes.
  • Talk about bigger issues collaboratively.
  • Marks Code Review task as done.

Why?

Mentoring

Polyglot

For the Children

For the Team

Thanks.

Engineering Review

By lectrotext

Engineering Review

  • 341
Loading comments...

More from lectrotext