|
1 | | ---- |
2 | | -name: ObjectFirst.Zabbix pull request template |
3 | | -about: This template is used when submitting a pull request for the ObjectFirst.Zabbix template. |
4 | | -title: '' |
5 | | -labels: '' |
6 | | -assignees: '' |
7 | | ---- |
| 1 | +# Pull Request Template |
8 | 2 |
|
9 | | -<!--- Provide a general summary of your changes in the Title above --> |
| 3 | +By contributing, you agree that your contributions will be licensed under the projects original open source license. |
10 | 4 |
|
11 | 5 | ## Description |
12 | 6 |
|
13 | | -<!--- Describe your changes in detail --> |
| 7 | +Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. |
14 | 8 |
|
15 | | -## Related Issue |
| 9 | +Fixes # (issue) |
16 | 10 |
|
17 | | -<!--- This project only accepts pull requests related to open issues --> |
18 | | -<!--- If suggesting a new feature or change, please discuss it in an issue first --> |
19 | | -<!--- If fixing a bug, there should be an issue describing it with steps to reproduce --> |
20 | | -<!--- Please link to the issue here: --> |
| 11 | +### Type of change |
21 | 12 |
|
22 | | -## Motivation and Context |
| 13 | +* [ ] Bug fix (non-breaking change which fixes an issue) |
| 14 | +* [ ] New feature (non-breaking change which adds functionality) |
| 15 | +* [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) |
| 16 | +* [ ] This change requires a documentation update |
23 | 17 |
|
24 | | -<!--- Why is this change required? What problem does it solve? --> |
| 18 | +### How Has This Been Tested? |
25 | 19 |
|
26 | | -## How Has This Been Tested? |
| 20 | +Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. |
27 | 21 |
|
28 | | -<!--- Please describe in detail how you tested your changes. --> |
29 | | -<!--- Include details of your testing environment, and the tests you ran to --> |
30 | | -<!--- see how your change affects other areas of the code, etc. --> |
| 22 | +### Checklist (check all applicable): |
31 | 23 |
|
32 | | -## Screenshots (if appropriate) |
33 | | - |
34 | | -## Types of changes |
35 | | - |
36 | | -<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> |
37 | | - |
38 | | -- [ ] Bug fix (non-breaking change which fixes an issue) |
39 | | -- [ ] New feature (non-breaking change which adds functionality) |
40 | | -- [ ] Breaking change (fix or feature that would cause existing functionality to change) |
41 | | - |
42 | | -## Checklist |
43 | | - |
44 | | -<!--- Go over all the following points, and put an `x` in all the boxes that apply. --> |
45 | | -<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> |
46 | | - |
47 | | -- [ ] My code follows the code style of this project. |
48 | | -- [ ] My change requires a change to the documentation. |
49 | | -- [ ] I have updated the documentation accordingly. |
50 | | -- [ ] I have read the **CONTRIBUTING** document. |
51 | | -- [ ] I have added tests to cover my changes. |
52 | | -- [ ] All new and existing tests passed. |
| 24 | +* [ ] My code follows the style guidelines of this project |
| 25 | +* [ ] I have performed a self-review of my own code |
| 26 | +* [ ] I have commented my code, particularly in _hard to understand_ areas |
| 27 | +* [ ] I have made corresponding changes to the documentation |
| 28 | +* [ ] My changes generate no new warnings |
| 29 | +* [ ] I have added tests that prove my fix is effective or that my feature works |
| 30 | +* [ ] New and existing unit tests pass locally with my changes |
0 commit comments