Confirmed users
673
edits
DevTools/ReviewTips: Difference between revisions
Jump to navigation
Jump to search
→Getting Review
No edit summary |
|||
| Line 67: | Line 67: | ||
** 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 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) | ** 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) | ||
* File followup bugs immediately after they are requested by reviewers, it's too easy to forget to do that. | |||
Other sources of hints and tips: | Other sources of hints and tips: | ||