forked from mozilla-mobile/firefox-ios
-
Notifications
You must be signed in to change notification settings - Fork 0
/
PULL_REQUEST_TEMPLATE
20 lines (13 loc) · 1.04 KB
/
PULL_REQUEST_TEMPLATE
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
*Thank you for submitting a pull request, your contributions are greatly appreciated!*
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
## Pull Request Checklist
- [ ] My patch has gone through review and I have addressed review comments
- [ ] My patch has a standard commit message that looks like `Bug 12345678 - This fixes something something`
- [ ] I have updated the *Unit Tests* to cover new or changed functionality
- [ ] I have updated the *UI Tests* to cover new or changed functionality
- [ ] I have marked the bug with `[needsuplift]`
- [ ] I have made sure that localizable strings use `NSLocalizableString()`
## Screenshots
If your patch includes user interface changes that you would like to suggest or that you would like UX to look at, please include them here.
## Notes for testing this patch
If useful, please leave notes for QA, explaining what this patch changes and how it can be best tested and verified.