www

commit af8c1307f7b0a1e6f07a33c006359dc94adff3c5
parent eef108b2ddb2f33da2d89cef39b01663f9a16b5f
Author: Emily Eisenberg <emily@khanacademy.org>
Date:   Wed, 22 Apr 2015 18:55:20 -0700

Add better lint checking.

Summary:
Make the lint checker check more files. Also, make arc run the
linter.

Test Plan:
 - `arc lint`
 - `make lint`

Reviewers: kevinb, alpert

Reviewed By: alpert

Differential Revision: https://phabricator.khanacademy.org/D17509

Diffstat:
M
.arcconfig
 | 
6
+++++-
M
Makefile
 | 
2
+-
M
contrib/auto-render/auto-render-spec.js
 | 
7
+++++++
M
contrib/auto-render/auto-render.js
 | 
5
+++--
M
server.js
 | 
8
++++----
M
test/katex-spec.js
 | 
6
++++++

6 files changed, 26 insertions(+), 8 deletions(-)
diff --git a/.arcconfig b/.arcconfig
@@ -1,4 +1,8 @@
 {
     "project_id": "KaTeX",
-    "conduit_uri": "https://phabricator.khanacademy.org/"
+    "conduit_uri": "https://phabricator.khanacademy.org/",
+    "lint.engine": "ArcanistSingleLintEngine",
+    "lint.engine.single.linter": "ArcanistScriptAndRegexLinter",
+    "linter.scriptandregex.regex": "/^(?P<file>\\S+): line (?P<line>\\d+), col \\d+, (?P<message>.*)$/m",
+    "linter.scriptandregex.script": "make lint || true"
 }
diff --git a/Makefile b/Makefile
@@ -12,7 +12,7 @@ export UGLIFYJS = $(realpath ./node_modules/.bin/uglifyjs) \
 setup:
 	npm install
 
-lint: katex.js $(wildcard src/*.js)
+lint: katex.js server.js cli.js $(wildcard src/*.js) $(wildcard test/*.js) $(wildcard contrib/*/*.js)
 	./node_modules/.bin/jshint $^
 
 build/katex.js: katex.js $(wildcard src/*.js)
diff --git a/contrib/auto-render/auto-render-spec.js b/contrib/auto-render/auto-render-spec.js
@@ -1,3 +1,9 @@
+/* global beforeEach: false */
+/* global jasmine: false */
+/* global expect: false */
+/* global it: false */
+/* global describe: false */
+
 var splitAtDelimiters = require("./splitAtDelimiters");
 
 beforeEach(function() {
@@ -28,6 +34,7 @@ beforeEach(function() {
                         var correct = result[i];
 
                         var good = true;
+                        var diff;
 
                         if (real.type !== correct.type) {
                             good = false;
diff --git a/contrib/auto-render/auto-render.js b/contrib/auto-render/auto-render.js
@@ -1,3 +1,5 @@
+/* global katex */
+
 var splitAtDelimiters = require("./splitAtDelimiters");
 
 var splitWithDelimiters = function(text, delimiters) {
@@ -48,9 +50,8 @@ var renderElem = function(elem, delimiters, ignoredTags) {
             if (shouldRender) {
                 renderElem(childNode, delimiters, ignoredTags);
             }
-        } else {
-            // Something else, ignore
         }
+        // Otherwise, it's something else, and ignore it.
     }
 };
 
diff --git a/server.js b/server.js
@@ -61,10 +61,10 @@ app.get("/katex.css", function(req, res, next) {
     });
 });
 
-app.use(express.static(path.join(__dirname, "static")));
-app.use(express.static(path.join(__dirname, "build")));
-app.use("/test", express.static(path.join(__dirname, "test")));
-app.use("/contrib", express.static(path.join(__dirname, "contrib")));
+app.use(express["static"](path.join(__dirname, "static")));
+app.use(express["static"](path.join(__dirname, "build")));
+app.use("/test", express["static"](path.join(__dirname, "test")));
+app.use("/contrib", express["static"](path.join(__dirname, "contrib")));
 
 app.use(function(err, req, res, next) {
     console.error(err.stack);
diff --git a/test/katex-spec.js b/test/katex-spec.js
@@ -1,3 +1,9 @@
+/* global beforeEach: false */
+/* global jasmine: false */
+/* global expect: false */
+/* global it: false */
+/* global describe: false */
+
 var buildHTML = require("../src/buildHTML");
 var buildMathML = require("../src/buildMathML");
 var katex = require("../katex");