- Does your code work, and the tests pass?
- Do you have a good description?
- Do you have a appropriate title with the correct prefix?
- Did you use the proper media type?
- Did you add a label and a project?
- Did you double check your code / comments?
Pull Request Checklist
This post is mainly for myself, to create a comprehensive quality checklist of things include/exclude from a PR.
Make sure the Patch compiles, test pass, and it does the thing it’s suppose to. It’s really easy to use the build bot to kick things off. Comment
@blender-bot packageon a PR to start builds for all supported platforms.
Look over files changes, and ensure it’s the changes you want. Double check all comments for spelling errors, and proper grammar.
Ensure the proper title prefix is used. Use the module prefix for features (I.e,
Fix #00000for bug fixes, the
#00000references a specific issue number, and will automatically close the issue when the PR is closed. Use
Cleanup:when either refactoring or cleaning up the code.
Add in a description of the problem that is addressed, a description of the proposed solution, and motivation.
Add the appropriate Label and Project to the PR.
Don’t use GIF’s, use MP4’s instead. This allows for vide scrubbing, pausing, etc. Ensure there is enough verbal description for the change to be indexed by a search engine.
Post PR’s out in the open. Keep things public.