-
Notifications
You must be signed in to change notification settings - Fork 11
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #22 from cyberark/add-pr-template-68
Add PR Template
- Loading branch information
Showing
1 changed file
with
21 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
### What does this PR do? | ||
- _What's changed? Why were these changes made?_ | ||
- _How should the reviewer approach this PR, especially if manual tests are required?_ | ||
- _Are there relevant screenshots you can add to the PR description?_ | ||
|
||
### What ticket does this PR close? | ||
Connected to #[relevant GitHub issues, eg 76] | ||
|
||
### Checklists | ||
|
||
#### Change log | ||
- [ ] The CHANGELOG has been updated, or | ||
- [ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update | ||
|
||
#### Test coverage | ||
- [ ] This PR includes new unit and integration tests to go with the code changes, or | ||
- [ ] The changes in this PR do not require tests | ||
|
||
#### Documentation | ||
- [ ] Docs (e.g. `README`s) were updated in this PR, and/or there is a follow-on issue to update docs, or | ||
- [ ] This PR does not require updating any documentation |