fix: add note about pr validation (#861)

* add note about pr validation

* Update PULL_REQUEST_TEMPLATE.md

* fix unrelated 301 test

.github/PULL_REQUEST_TEMPLATE.md

@@ -45,6 +45,7 @@
 ## Added to documentation?
 
 - [ ] 📜 readme
+- [ ] 📜 contributing.md
 - [ ] 🙅 no documentation needed
 
 ## [optional] Are there any post-deployment tasks we need to perform?

CONTRIBUTING.md

@@ -50,6 +50,20 @@ $ npm run test
 
 *note for maintainers: All pull requests need a label to assist automation. See the [template](https://github.com/open-sauced/open-sauced/blob/HEAD/.github/release-drafter.yml) to guide which labels to use.*
 
+### PR Validation
+Examples for valid PR titles:
+
+- fix: Correct typo.
+- feat: Add support for Node 12.
+- refactor!: Drop support for Node 6.
+
+_Note that since PR titles only have a single line, you have to use the ! syntax for breaking changes._
+
+See [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/) for more examples.
+
+### Work in progress
+Github has support for draft pull requests, which will disable the merge button until the PR is marked as ready for merge.
+
 ## Issues
 
 If you plan to contribute a change based on an open issue, please assign yourself by commenting on the following word `.take`. Issues that are not assigned are assumed open, and to avoid conflicts, please assign yourself before beginning work on any issues.

src/lib/repoStatusCode.test.js

@@ -13,9 +13,9 @@ describe("Test: repoStatusCode()", () => {
     expect(result).toBe(404);
   });
 
-  test("bdougie/open-sauced should return 301", async() => {
+  test("bdougie/open-sauced should return 200", async() => {
     const repoUrl = "bdougie/open-sauced";
     const result = await repoStatusCode(repoUrl);
-    expect(result).toBe(301);
+    expect(result).toBe(200);
   });
 });