Quality Stage 4 · Build sprints · Months 4–14

Controlling change during build sprints

During build sprints every code change goes through a pull request that a second engineer reviews against a checklist before it merges, and automated tests must pass first. Problems found along the way go into an anomaly log with a link to risk. A short design review at the end of each sprint confirms that requirements, design documents and tests moved with the code. Training records show each person was trained on the procedures before doing the work.

On the map: Change control and code review. Anomaly log. Design review each sprint. Training records.

All 40 tasks

Rows are disciplines, columns are stages 1 to 8. Select a square to open that task.

Back to the full map
A saved reply in GitHub turns the development plan's review checklist into a one-click comment on each pull request.
A saved reply in GitHub turns the development plan's review checklist into a one-click comment on each pull request. Source: How to Grow a 10x Team with Code Reviews.

Who you work with

Make code review and change control fast enough that engineering uses them every sprint. Anomalies logged when found are cheaper than anomalies reconstructed at verification.

Decisions you make

What does the code reviewer have to check?

A fixed checklist taken from the development plan: the requirement in the issue is implemented, the change fits the documented architecture, design documents are updated, and any requirement met by the change has a verification protocol. Saved replies in the review tool make the checklist cheap to apply on every pull request.

Who reviews high-risk code?

Senior engineers review changes to the highest safety class software items. A code owners file can force a named reviewer on files that implement risk controls.

When does a bug become a logged anomaly?

Any behavior that deviates from requirements, design documents or user expectations is logged, with its source, safety impact and security impact. Fixed items move to the version history. Open items carry forward to the unresolved anomalies list for the submission.

How heavy is the per-sprint design review?

Light. It confirms the sprint's outputs match its inputs and that documents moved with the code. Formal phase reviews with an independent reviewer stay at the phase gates.

Read more

Want to do this kind of work?

See the open positions and apply.

Apply Now
Apply now →