-
Notifications
You must be signed in to change notification settings - Fork 170
Reviewing a Pull request
Steve Crawford edited this page Aug 6, 2018
·
3 revisions
- Does the code following PEP8 with allowed exceptions?
- Are the naming conventions for object followed?
- Does all new functionality have docstrings?
- If the PR closes an issue, does it resolve the matter raised?
- Does it include a change log entry?
-
Has the documentation been updated to include a description of the functionality?
-
Has the documentation been updated to include a description of reference files needed?
-
Has a unit test been added if appropriate?
-
Has a regression test been added if appropriate?
- Has a test been added to catch the bug in the future?