Confirmed users
811
edits
m (→Criteria to request approval (Guidelines): typo fix) |
Joshwalker (talk | contribs) No edit summary |
||
| Line 13: | Line 13: | ||
= Guidelines on approval comments = | = Guidelines on approval comments = | ||
* [Feature/regressing bug #]: If | * [Feature/regressing bug #]: If this is a fallout from a feature/bug, or a backout please state the reason along with the bug number | ||
* [User impact if declined]: In addition to the STR (steps to reproduce) reported in the bug if you can explain on a deeper level aspect of how an end user would be impacted with/without your change | * [User impact if declined]: In addition to the STR (steps to reproduce) reported in the bug if you can explain on a deeper level aspect of how an end user would be impacted with/without your change | ||
* [Describe test coverage new/current, TBPL]: What kind of testing was completed here. Manual and/or automated ? Any details about what scenarios the tests cover? If the patch landed on a master/central branch or any other branch did we get verification from reporter or QA. | * [Describe test coverage new/current, TBPL]: What kind of testing was completed here. Manual and/or automated ? Any details about what scenarios the tests cover? If the patch landed on a master/central branch or any other branch did we get verification from reporter or QA. | ||