-
Notifications
You must be signed in to change notification settings - Fork 50
Checklist
Christina-Kang edited this page Apr 5, 2018
·
19 revisions
This checklist lists items to check before submitting a pull request:
- Ensure that the PR has a descriptive summary.
- Update the
<root>\CODEOWNERS
file as appropriate. Instruction for how to add is included in the file itself. - Update the
<root>\src\README.rst
file to summarize the changes included in the pull request. If required by semantic versioning, include change logs under a new version. Include the PR number at the end of each entry, with formatNew feature (#70)
where 70 is the pull request number. - In the
<root>\src\README.rst
file, all new entries that are not published yet should be marked as 'Unreleased'. - For each new command added, make sure appropriate documentation is added to the corresponding help file.
- If your command has a custom help text section, check that the content is still current.
- Any new functionality should include testing. Tests that require a live cluster connection are optional, and should be avoided if possible. Non live testing is required. Suggested testing includes unit testing, as well as paths generation test, which can be found in
<root>\src\sfctl\tests\request_generation_test.py
. - Any new commands, command groups, or commands whose names have been changed, should include a test in the file
<root>\src\sfctl\tests\help_text_test.py
. This test helps ensure that the help text is present and displayed correctly. - Make sure that all test assert calls include an explanatory message for ease of debugging errors.
- Ensure that all new code follows PEP8 conventions (https://www.python.org/dev/peps/pep-0008/).
- Development
- How-To
- Details
- Service Fabric Developers