Merge pull request #46 from aicore/pr

Adding pull request template

Author: abose

.github/ISSUE_TEMPLATE/pull_request_template.md renamed to .github/PULL_REQUEST_TEMPLATE/bug_fix.md

@@ -1,18 +1,24 @@
+---
+name: Bug fix
+about: Change which fixes an issue.
+title: ''
+labels: ''
+assignees: ''
+
+---
 # Description
 
-* Please include a summary of the change and which issue is addressed. 
+* Please include a summary of the change. 
 * Please also include relevant motivation and context. 
 * List any dependencies that are required for this change.
 
-Fixes # (issue)
+Addresses issues #(issue)
 
 ## Type of change
 
-Please delete options that are not relevant.
+**NB: ~~Please strike through options that are not applicable.~~**
 
-- [ ] 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 not work as expected)
+- [ ] Breaking change (fix that would cause existing functionality to not work as expected)
 - [ ] This change requires a documentation update
 
 # Sample Input and output
@@ -21,7 +27,10 @@ Please delete options that are not relevant.
 
 # How Has This Been Tested?
 
-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
+* 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
+
 
 - [ ] Test A
 - [ ] Test B
@@ -34,7 +43,7 @@ Please describe the tests that you ran to verify your changes. Provide instructi
 
 # Checklist:
 
-Add N/A to items thata re not applicanble.
+**NB: ~~Please strike through options that are not applicable.~~**
 
 - [ ] My code follows the style guidelines of this project
 - [ ] I have performed a self-review of my own code

.github/PULL_REQUEST_TEMPLATE/new_feature.md

@@ -0,0 +1,57 @@
+---
+name: New Feature
+about: Change which adds functionality
+title: ''
+labels: ''
+assignees: ''
+---
+
+# Description
+
+* Please include a summary of the change. 
+* Please also include relevant motivation and context. 
+* List any dependencies that are required for this change.
+
+Fixes #(issue)
+
+## Type of change
+**NB: ~~Please strike through options that are not applicable.~~**
+- [ ] non-breaking change which fixes an issue
+- [ ] Breaking change (feature that would cause existing functionality to not work as expected)
+- [ ] This change requires a documentation update
+
+# Sample Input and output
+* Provide screenshots or videos.
+* Or provide code block Input output if run on text terminal as below.
+```
+  $ code block Input output on text terminal.
+  output
+```
+
+# How Has This Been Tested?
+
+* 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
+
+- [ ] Test A
+- [ ] Test B
+
+**Test Configuration**:
+* Firmware version:
+* Hardware:
+* Toolchain:
+* SDK:
+
+# Checklist:
+
+**NB: ~~Please strike through options that are not applicable.~~**
+
+- [ ] My code follows the style guidelines of this project
+- [ ] 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
+- [ ] My changes generate no new warnings
+- [ ] I have added tests that prove my fix is effective or that my feature works
+- [ ] New and existing unit tests pass locally with my changes
+- [ ] Any dependent changes have been merged and published in downstream modules

.github/PULL_REQUEST_TEMPLATE/remove_feature.md

@@ -0,0 +1,58 @@
+---
+name: Remove Feature
+about: Change which removes existing functionality
+title: ''
+labels: ''
+assignees: ''
+---
+
+# Description
+
+* Please include a summary of the change. 
+* Please also include relevant motivation and context. 
+* List any dependencies that are required for this change.
+
+Fixes #(issue)
+
+## Type of change
+**NB: ~~Please strike through options that are not applicable.~~**
+- [ ] non-breaking change which fixes an issue
+- [ ] Breaking change (feature that would cause existing functionality to not work as expected)
+- [ ] This change requires a documentation update
+
+# Sample Input and output
+* Provide screenshots or videos.
+* Or provide code block Input output if run on text terminal as below.
+```
+  $ code block Input output on text terminal.
+  output
+```
+
+# How Has This Been Tested?
+
+* 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
+
+
+- [ ] Test A
+- [ ] Test B
+
+**Test Configuration**:
+* Firmware version:
+* Hardware:
+* Toolchain:
+* SDK:
+
+# Checklist:
+
+**NB: ~~Please strike through options that are not applicable.~~**
+
+- [ ] My code follows the style guidelines of this project
+- [ ] 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
+- [ ] My changes generate no new warnings
+- [ ] I have added tests that prove my fix is effective or that my feature works
+- [ ] New and existing unit tests pass locally with my changes
+- [ ] Any dependent changes have been merged and published in downstream modules