Update PULL_REQUEST_TEMPLATE.md (#23)

### Summary of Changes

This PR consolidates the pull request template initially added by @verdan. It's been requested to be updated across all of our repos: amundsen-io/amundsenfrontendlibrary#125 (comment)

### Tests

Unit tests not required for template changes.

### Documentation

Documentation not required for template changes.

### CheckList
Make sure you have checked **all** steps below to ensure a timely review.
- [ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"
    - In case you are adding a dependency, check if the license complies with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
- [ ] PR includes a summary of changes. 
- [ ] PR adds unit tests, updates existing unit tests, __OR__ documents why no test additions or modifications are needed.
- [ ] In case of new functionality, my PR adds documentation that describes how to use it.
    - All the public functions and the classes in the PR contain docstrings that explain what it does
- [ ] PR passes `make test`
- [ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)"

Author: ttannis

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,34 +1,22 @@
-Make sure you have checked **all** steps below.
+### Summary of Changes
 
-### Title
-
-- [ ] My PR Title addresses the issue accurately and concisely. 
-    - Example: "Updates the version of Flask to v1.0.2"
-    - In case you are adding a dependency, check if the license complies with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
-
-### Description
-
-- [ ] Here are some details about my PR:
+_Include a summary of changes then remove this line_
 
 ### Tests
 
-- [ ] My PR adds the following unit tests __OR__ does not need testing for this extremely good reason:
-
-### Commits
-
-- [ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)":
-    1. Subject is separated from body by a blank line
-    1. Subject is limited to 50 characters (not including Jira issue reference)
-    1. Subject does not end with a period
-    1. Subject uses the imperative mood ("add", not "adding")
-    1. Body wraps at 72 characters
-    1. Body explains "what" and "why", not "how"
+_What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line_
 
 ### Documentation
 
+_What documentation did you add or modify and why? Add any relevant links then remove this line_
+
+### CheckList
+Make sure you have checked **all** steps below to ensure a timely review.
+- [ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"
+    - In case you are adding a dependency, check if the license complies with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
+- [ ] PR includes a summary of changes. 
+- [ ] PR adds unit tests, updates existing unit tests, __OR__ documents why no test additions or modifications are needed.
 - [ ] In case of new functionality, my PR adds documentation that describes how to use it.
     - All the public functions and the classes in the PR contain docstrings that explain what it does
-
-### Code Quality & Coverage
-
-- [ ] Passes `make test`
+- [ ] PR passes `make test`
+- [ ] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)"