2079
Comment: formatting
|
2097
Bug 669936 will not be fixed, so self-review without review type.
|
Deletions are marked like this. | Additions are marked like this. |
Line 39: | Line 39: |
Submit the branch to create an MP (our toolchains can look at this and it provides a location for a post landing review if the branch has that done to it). Self review with review type 'unreviewed'. Land via the normal landing process. | 1. Submit the branch to create an MP ''(our toolchains can look at this and it provides a location for a post landing review if the branch has that done to it.)'' 1. Self review ''without'' a review type. 1. Land via the normal landing process. |
Name: Optional Reviews
Owner: Technical Architect
Effective: Not yet
Review: (When the page describes an experiment, when it should be reviewed)
Experiment Underway:
Process Overview
Some reviews are not needed and eligible developers can choose to have branches land without formal review.
Process Description
Developers which are contributing at or near fulltime to Launchpad can, after 3 months, choose to land branches without review.
Each month, all code reviewers will be assigned one unreviewed landed to do a post hoc review as part of assessing the experiment.
Production problems which are tracked to unreviewed landings will also be input into assessing the experiment.
Rationale
Not all changes benefit from code review / are high enough risk to need formal review. For instance (not exhaustive):
- mechanical things (like moving code)
- updating source deps
- rollbacks
- typo fixes
- improvements to documentation
For many of these things a review may add value - but less value than doing the review uses up. We want reviews to be a net win for the effort being put into developing Launchpad, and so we should, once we're comfortable people know how things should be, allow them to decide if a particular change is an improvement on its own, or an improvement that also /needs/ other input before landing.
Triggers
Experienced developer (someone currently working on LP who has been doing so for the last 3 months) landing a code change (not UI or Database, for now : evaluate after assessing the success of optional code reviews).
Activities
Submit the branch to create an MP (our toolchains can look at this and it provides a location for a post landing review if the branch has that done to it.)
Self review without a review type.
- Land via the normal landing process.