fixed some linting rules and updated some instructions. More to come.

ryan-hamblin · ryan-hamblin · commit e8016df6666b · 2018-02-13T14:50:12.000-07:00
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -13,6 +13,7 @@
     "arrow-body-style": 0,
     "no-unused-vars": 0,
     "no-useless-constructor": 0,
-    "import/no-unresolved": 0
+    "import/no-unresolved": 0,
+    "comma-dangle": "off"
   }
 }
diff --git a/README.md b/README.md
@@ -1,13 +1,24 @@
 # JavaScript I
 
-## Instructions
+## Project initialization
 
-Do not use any of the built in Javascript Array methods like .forEach .map etc. If you'd like feel free to reuse any of your functions along the way though! 
-
-#### Fork and clone this repo.
-* Clone this repository and navigate into it.
+* `Fork and Clone` this repository and cd into it.
 * Run the command `npm i` to install needed node packages.
-* Run the command `npm test` to run the tests.
-* Work through the files and make the tests pass.
-* Suggested order: `callbacks.js`, `arrays.js`, `objects.js`, `closure.js`.
-* Submit a pull request when you are finished and we will review your code.
+* **JUST** a _reminder_ this command runs a script on your machine based on the `package.json` file you have installed in your project. By running this command, you're telling your project to head over to `npm.org` and download all of the packages you need for this project.
+  ## Instructions
+* This project is designed to carry over from JavaScript I through JavaScript II.
+* Please complete the project files in this order: `variables.js` `callbacks.js` `arrays.js` `objects.js` and finally `closure.js`.
+* In each project file you'll see a series of problems to work through,.
+* When you feel like you've completed the problem, be sure to run, `npm test <file name>` to get see if your code works properly... and to check if you have any linting errors.
+* Work until you have all of the tests passing.
+  ## Running the tests
+  Example:
+
+```
+npm test arrays
+
+-> executes all of the tests in the `src/tests/arrays.tests.js file`
+
+npm test closure
+-> executes all of the tests in the `src/tests/closure.tests.js file`
+```
diff --git a/src/callbacks.js b/src/callbacks.js
@@ -1,5 +1,3 @@
-/* eslint-disable */
-
 const firstItem = (arr, cb) => {
   // firstItem passes the first item of the given array to the callback function.
 };
@@ -41,5 +39,5 @@ module.exports = {
   sumNums,
   multiplyNums,
   contains,
-  removeDuplicates,
+  removeDuplicates
 };
diff --git a/src/variables.js b/src/variables.js
@@ -0,0 +1 @@
+// TODO: fill this in with some awesome variable examples using let, const and var.
diff --git a/tests/variables.test.js b/tests/variables.test.js

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@`
`13`	`13`	`"arrow-body-style": 0,`
`14`	`14`	`"no-unused-vars": 0,`
`15`	`15`	`"no-useless-constructor": 0,`
`16`		`- "import/no-unresolved": 0`
	`16`	`+ "import/no-unresolved": 0,`
	`17`	`+ "comma-dangle": "off"`
`17`	`18`	`}`
`18`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+// TODO: fill this in with some awesome variable examples using let, const and var.`