Linting (#29)

* Added linting checks

* Fixed linting feedback

* I was hoping linting would actually find this error... but alas

* AppVeyor did not like the combined test command

appveyor.yml

@@ -15,7 +15,7 @@ test_script:
   - node --version
   - npm --version
   # run tests
-  - npm test -- -t 10000
+  - npm test -- -- -t 10000 #the extra -- is because npm test runs multiple script commands
 
 # Don't actually build.
 build: off

package.json

@@ -27,9 +27,19 @@
   "devDependencies": {
     "chai": "^3.5.0",
     "coffee-script": "^1.10.0",
+    "coffeelint": "^1.16.0",
     "mocha": "^3.0.2"
   },
   "scripts": {
-    "test": "mocha --compilers coffee:coffee-script/register"
+    "test": "npm run lint & npm run mocha",
+    "mocha": "mocha --compilers coffee:coffee-script/register",
+    "lint": "coffeelint lib test"
+  },
+  "coffeelintConfig": {
+    "max_line_length": {
+      "value": 100,
+      "level": "error",
+      "limitComments": true
+    }
   }
 }

test/empty_repo.coffee

@@ -19,5 +19,5 @@ describe 'Constructing hg-repository', ->
     repo = new HgRepository testRepo.fullPath()
     assert.ok repo
 
-  after ->    
+  after ->
     testRepo.destroy()

test/fakeWindow.coffee

@@ -1,4 +1,4 @@
 global.window = {
-    addEventListener: ()->
-    removeEventListener: ()->
+  addEventListener: ()->
+  removeEventListener: ()->
 }

test/testRepository.coffee

@@ -21,9 +21,9 @@ module.exports =
       path.join __dirname, 'test_repo'
 
     exists: () ->
-      fs.existsSync(@fullPath)
+      fs.existsSync(@fullPath())
 
-    destroy: ->      
+    destroy: ->
       if isWindows
         exec 'powershell -command "remove-item -recurse -force ' + @fullPath() + '"'
       else