Merge pull request #56 from codeclimate/will/eslintrc

Add app's eslintrc

Author: wfleming

.eslintrc

@@ -3,6 +3,25 @@
     "node": true
   },
   "rules": {
+    "brace-style": [2, "1tbs", { "allowSingleLine": true }],
+    "comma-dangle": [2, "never"],
+    "comma-style": [2, "first", { exceptions: {ArrayExpression: true, ObjectExpression: true} }],
+    "curly": 2,
+    "eqeqeq": [2, "allow-null"],
+    "no-shadow-restricted-names": 2,
+    "no-undef": 2,
+    "no-use-before-define": 2,
+    "radix": 2,
+    "semi": 2,
+    "space-infix-ops": 2,
     "strict": 0
+  },
+  "globals": {
+    "AnalysisView": true,
+    "PollingView": true,
+    "Prism": true,
+    "Spinner": true,
+    "Timer": true,
+    "moment": true
   }
 }

bin/eslint.js

@@ -18,13 +18,18 @@ var checks = require("../lib/checks");
 
 // a wrapper for emitting perf timing
 function runWithTiming(name, fn) {
-  var start = new Date(),
-      rv = fn(),
-      duration = (new Date() - start) / 1000;
+  var start = new Date()
+    , rv = fn()
+    , duration = (new Date() - start) / 1000;
   console.error("eslint.timing." + name + ": " + duration + "s");
   return rv;
 }
 
+function contentBody(check) {
+  var content = docs.get(check) || "For more information visit ";
+  return content + "Source: http://eslint.org/docs/rules/\n";
+}
+
 function buildIssueJson(message, path) {
   // ESLint doesn't emit a ruleId in the
   // case of a fatal error (such as an invalid
@@ -60,11 +65,6 @@ function buildIssueJson(message, path) {
   return JSON.stringify(issue);
 }
 
-function contentBody(check) {
-  var content = docs.get(check) || "For more information visit ";
-  return content + "Source: http://eslint.org/docs/rules/\n";
-}
-
 function isFileWithMatchingExtension(file, extensions) {
   var stats = fs.lstatSync(file);
   var extension = "." + file.split(".").pop();
@@ -79,7 +79,7 @@ function isFileIgnoredByLibrary(file) {
   var path = file.replace(/^\/code\//, "");
   var ignored = cli.isPathIgnored(path);
   if (ignored) {
-    output = "File `" + path + "` ignored because of your .eslintignore file." + "\n";
+    var output = "File `" + path + "` ignored because of your .eslintignore file." + "\n";
     process.stderr.write(output);
   }
   return ignored;
@@ -94,7 +94,7 @@ function prunePathsWithinSymlinks(paths) {
   return paths.filter(function(path) {
     var withinSymlink = false;
     symlinks.forEach(function(symlink) {
-      if (path.indexOf(symlink) == 0) {
+      if (path.indexOf(symlink) === 0) {
         withinSymlink = true;
       }
     });
@@ -193,10 +193,10 @@ var analysisFiles = runWithTiming("buildFileList", function() {
 });
 
 function analyzeFiles() {
-  var batchNum = 0,
-      batchSize = 1,
-      batchFiles,
-      batchReport;
+  var batchNum = 0
+    , batchSize = 1
+    , batchFiles
+    , batchReport;
 
   while(analysisFiles.length > 0) {
     batchFiles = analysisFiles.splice(0, batchSize);

lib/checks.js

@@ -110,17 +110,18 @@ var checkCategoryMapping = {
 };
 
 var categories = function(checkName) {
-  return [checkCategoryMapping[checkName] || "Style"]
+  return [checkCategoryMapping[checkName] || "Style"];
 };
 
 var remediationPoints = function(checkName, message) {
   if (checkName === "complexity") {
     // (@base_cost + (overage * @cost_per))*1_000_000
     // cost_per: 0.1,  base: 1
-    var costPer = 100000,
-        points = 1000000,
-        threshold = 10, // TODO: Get this from the eslintrc
-        overage, complexity;
+    var costPer = 100000
+      , points = 1000000
+      , threshold = 10 // TODO: Get this from the eslintrc
+      , overage
+      , complexity;
 
     complexity = message.message.match(/complexity of (\d+)/)[1];
     overage = complexity - threshold;