Hui Zhou
8fc3c0a87d
Having too many checklist items are not effective. Combine some of them and omit those we automatically enforce using github merge policy.
757 B
757 B
Pull Request Description
Author Checklist
- Provide Description Particularly focus on why, not what. Reference background, issues, test failures, xfail entries, etc.
- Commits Follow Good Practice
Commits are self-contained and do not do two things at once.
Commit message is of the form:
module: short descriptionCommit message explains what's in the commit. - Passes All Tests Whitespace checker. Warnings test. Additional tests via comments.
- Contribution Agreement For non-Argonne authors, check contribution agreement. If necessary, request an explicit comment from your companies PR approval manager.