Confirmed users
295
edits
DevTools/ReviewTips: Difference between revisions
→Getting Review
| Line 64: | Line 64: | ||
** A patch that cleanly applies to fx-team | ** A patch that cleanly applies to fx-team | ||
** A patch with a message in the format 'Bug XXXXXX - Message' (pro-tip: cut and paste from the Bug page) | ** A patch with a message in the format 'Bug XXXXXX - Message' (pro-tip: cut and paste from the Bug page) | ||
** The | ** The bug's 'depends' field should correctly list any bugs that should be committed before this one (some reviewers test the patch, and you'll need it to land anyway) | ||
Other sources of hints and tips: | Other sources of hints and tips: | ||