Something went wrong on our end
-
magnus2142 authored3a845c34
PULL_REQUEST_TEMPLATE.md 1.44 KiB
[Short description explaining the high-level reason for the pull request]
Additions
Removals
Changes
Testing
Screenshots
Notes
Todos
Checklist
- PR has an informative and human-readable title
- Changes are limited to a single goal (no scope creep)
- Code can be automatically merged (no conflicts)
- Code follows the standards laid out in the development playbook
- Passes all existing automated tests
- Any change in functionality is tested
- New functions are documented (with a description, list of inputs, and expected output)
- Placeholder code is flagged / future todos are captured in comments
-
Visually tested in supported browsers and devices (see checklist below
) - Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
-
Reviewers requested with the Reviewers tool
Testing checklist
Browsers
- Chrome
- Firefox
- Safari
- Internet Explorer 8, 9, 10, and 11
- Edge
- iOS Safari
- Chrome for Android
Accessibility
- Keyboard friendly
- Screen reader friendly
Other
- Is useable without CSS
- Is useable without JS
- Flexible from small to large screens
- No linting errors or warnings
- JavaScript tests are passing