-
-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add pull request template for PlaceCal (#2630)
- Loading branch information
1 parent
3f91dc7
commit cdba08e
Showing
1 changed file
with
53 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,53 @@ | ||
<!--- Provide a general summary of your changes in the Title above --> | ||
<!--Read comments, before committing pull request read checklist again | ||
# Checklist: | ||
- [ ] I have performed a self-review of my own code, | ||
- [ ] I have commented my code, particularly in hard-to-understand areas, | ||
- [ ] I have made corresponding changes to the documentation, | ||
- [ ] I have added tests that prove my fix is effective or that my feature works, | ||
- [ ] Title include "WIP" if work is in progress. | ||
--> | ||
|
||
Resolves # <!--fill issue number--> | ||
|
||
## Description | ||
|
||
<!--- Describe your changes in detail --> | ||
|
||
### Motivation and Context | ||
|
||
<!--- Why is this change required? What problem does it solve? --> | ||
<!--- If it fixes an open issue, please link to the issue here. --> | ||
|
||
<!--- If there's a Figma file or similar spec, please link to to the issue. List any dependencies that are required for this change. (gems, js libraries, etc.) | ||
Include anything else we should know about.--> | ||
|
||
### Type of change | ||
|
||
<!-- Please delete options that are not relevant. --> | ||
|
||
- [ ] Bug fix (non-breaking change which fixes an issue) | ||
- [ ] New feature (non-breaking change which adds functionality) | ||
- [ ] Breaking change (fix or feature that would cause existing functionality to change) | ||
- [ ] This change requires a change to the deployment process | ||
- [ ] This change requires a documentation update | ||
- [ ] Documentation update | ||
|
||
### How Has This Been Tested? | ||
|
||
<!--- Please describe in detail how you tested your changes so that a reviewer can reproduce the results. --> | ||
<!--- Include details of your testing environment, and the tests you've run your self --> | ||
<!--- see how your change affects other areas of the code, etc. --> | ||
|
||
### How Will This Be Deployed? | ||
|
||
<!--- Run through any change to infrastructure, CLI commands or manual admin config to do --> | ||
<!--- For example, any env variables to add to the hosting infra? --> | ||
|
||
### Screenshots | ||
|
||
<!--Optional. Delete if not relevant. | ||
Include screenshots (before / after) for style changes, highlight edited element.--> |