Add a bunch of eslint-disable comments for legacy code.

Summary: We're introducing a new, stricter js/jsx linter in the khan-linter repo: eslint. It has a bunch of rules to enforce our style guide, that the old jshint did not have. But most legacy code doesn't follow our style guide so well. This change allows us to upgrade to the new linter for new code, while keeping old code compliant. I do this by adding 'eslint-disable' directives everywhere. Here's the mondo command I ran to do this all automatically. First, I applied D22374 in my khan/devtools/khan-linter rule. Then, in khan-exercises/, I ran this mondo command: ~/khan/devtools/khan-linter/runlint.py > /var/tmp/le cat /var/tmp/le | grep -v 'File ignored' | perl -nle '/^([^:]*):\d+:\d+: [EW]([^ ]*)/ and print "$1 $2"' | sort -u | perl -ane '$t{$F[0]} .= " $F[1],"; if (eof) {for $k (keys %t) { chop $t{$k}; open(OLD, $k); open(NEW, ">$k.tmp"); print NEW qq{/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */\n/* eslint-disable$t{$k} */\n/* To fix, remove an entry above, run ka-lint, and fix errors. */\n\n}; while (<OLD>) {print NEW $_}; close(OLD); close(NEW); rename("$k.tmp", "$k") }}' It's not as bad as it looks. Basically it runs the linter, munges the output to be 'filename violated-rule-name' pairs, and then uses some perl to collect all the violated-rules for a single filename in one place, and to prepend the appropriate eslint-disable comment on to the file. Test Plan: In khan-exercises/, ran ~/khan/devtools/khan-linter/runlint.py and saw no errors. Reviewers: emily, kevinb Subscribers: john Differential Revision: https://phabricator.khanacademy.org/D22393
Khan · Oct 13, 2015 · cf7b219 · cf7b219
1 parent 4e8bdb3
commit cf7b219
Show file tree

Hide file tree

Showing 64 changed files with 256 additions and 0 deletions.
diff --git a/build/calculator/calculator-tail.js b/build/calculator/calculator-tail.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, eqeqeq, max-len, no-undef, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 window.Calculator = (function(parser) {
     // I18N: calculator error message
     var ERROR_TEXT = $._("Error");

diff --git a/build/kathjax-config.js b/build/kathjax-config.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, eqeqeq, indent, max-len, no-unused-vars, one-var, prefer-template, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 MathJax.Hub.Config({
     messageStyle: "none",
     skipStartupTypeset: true,

diff --git a/build/pack.js b/build/pack.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, max-len, no-console, prefer-template, space-before-function-paren */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*jshint node:true */
 /**
  * A node.js script that packs javascript in exercise files.

diff --git a/exercises-stub.js b/exercises-stub.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /**
  * Stub version of the Exercises object used in the live Khan Academy site.
  */

diff --git a/history.js b/history.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-undef, one-var, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /**
  * Event handlers for when problem history is being viewed.
  */

diff --git a/interface.js b/interface.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, eqeqeq, indent, max-len, no-undef, one-var, prefer-template, space-after-keywords, space-before-function-paren */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /**
  * Interface glue to handle events from 'Exercises' and talk to 'Khan' or some
  * Perseus object, whichever is appropriate for the current exercise.

diff --git a/khan-exercise.js b/khan-exercise.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-console, no-trailing-spaces, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-before-blocks */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /* khan-exercise.js
 
     The main entry point here is essentially the onjQueryLoaded method around

diff --git a/utils/algebra-intuition.js b/utils/algebra-intuition.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-undef, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 KhanUtil.scale = {

diff --git a/utils/angles.js b/utils/angles.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/answer-types.js b/utils/answer-types.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-after-keywords */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 var MAXERROR_EPSILON = Math.pow(2, -42);

diff --git a/utils/chemistry.js b/utils/chemistry.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-undef */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math-format.js");

diff --git a/utils/cipher.js b/utils/cipher.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, indent, max-len, no-redeclare, no-unused-vars, one-var */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/congruency.js b/utils/congruency.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./angles.js");

diff --git a/utils/constructions.js b/utils/constructions.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, prefer-template, space-before-function-paren */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 var kmatrix = require("./kmatrix.js");

diff --git a/utils/convert-values.js b/utils/convert-values.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, indent, max-len, no-redeclare, no-trailing-spaces, no-unused-vars, prefer-template, space-after-keywords, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/crc32.js b/utils/crc32.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable eqeqeq, indent, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 /*

diff --git a/utils/derivative-intuition.js b/utils/derivative-intuition.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-undef, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("../third_party/jquery.mobile.vmouse.js");

diff --git a/utils/exponents.js b/utils/exponents.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable camelcase, comma-dangle, indent, max-len, no-undef, one-var, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math.js");

diff --git a/utils/expressions.js b/utils/expressions.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-undef, no-unused-vars, one-var, prefer-template, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/functional.js b/utils/functional.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/geom.js b/utils/geom.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, no-undef, no-unused-vars, one-var */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 var eps = 1e-9;

diff --git a/utils/graphie-3d.js b/utils/graphie-3d.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, no-undef, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./graphie.js");

diff --git a/utils/graphie-drawing.js b/utils/graphie-drawing.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-undef */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math.js");

diff --git a/utils/graphie-geometry.js b/utils/graphie-geometry.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, comma-spacing, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var, prefer-template, space-before-keywords, space-infix-ops, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./graphie.js");

diff --git a/utils/graphie-helpers-arithmetic.js b/utils/graphie-helpers-arithmetic.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, indent, max-len, no-redeclare, no-trailing-spaces, no-undef, no-unused-vars, one-var, prefer-template, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 var decimalPointSymbol = icu.getDecimalFormatSymbols().decimal_separator;

diff --git a/utils/graphie-helpers.js b/utils/graphie-helpers.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, indent, max-len, no-redeclare, no-trailing-spaces, no-undef, one-var, prefer-spread, prefer-template, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math-format.js");

diff --git a/utils/graphie-polygon.js b/utils/graphie-polygon.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/graphie.js b/utils/graphie.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable brace-style, comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 var kpoint = require("./kpoint.js");

diff --git a/utils/interactive.js b/utils/interactive.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-infix-ops, space-unary-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("../third_party/jquery.mobile.vmouse.js");

diff --git a/utils/jquery.adhesion.js b/utils/jquery.adhesion.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, max-len */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 (function($) {
     $.fn.adhere = function(options) {
         options = $.extend({

diff --git a/utils/khanscript.js b/utils/khanscript.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable indent, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.fn.khanscript = function(problem) {

diff --git a/utils/kline.js b/utils/kline.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-trailing-spaces, no-undef, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * Line Utils
  * A line is an array of two points e.g. [[-5, 0], [5, 0]].

diff --git a/utils/kmatrix.js b/utils/kmatrix.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, indent, max-len, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-before-blocks, space-before-function-paren, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./expressions.js");

diff --git a/utils/knumber.js b/utils/knumber.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable camelcase, comma-dangle, eqeqeq, indent, no-undef, no-unused-vars, one-var, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * Number Utils
  * A number is a js-number, e.g. 5.12

diff --git a/utils/kpoint.js b/utils/kpoint.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-undef */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * Point Utils
  * A point is an array of two numbers e.g. [0, 0].

diff --git a/utils/kray.js b/utils/kray.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, indent */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * Ray Utils
  * A ray is an array of an endpoint and another point along the ray.

diff --git a/utils/kvector.js b/utils/kvector.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-trailing-spaces, no-undef, no-unused-vars, prefer-spread */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * Vector Utils 
  * A vector is an array of numbers e.g. [0, 3, 4].

diff --git a/utils/math-format.js b/utils/math-format.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-throw-literal, no-trailing-spaces, no-undef, one-var, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math.js");

diff --git a/utils/math.js b/utils/math.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, comma-spacing, eqeqeq, indent, max-len, no-constant-condition, no-redeclare, no-undef, no-unused-vars, one-var, prefer-spread, prefer-template, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 // Minify Raphael ourselves because IE8 has a problem with the 1.5.2 minified release

diff --git a/utils/matrix-input.js b/utils/matrix-input.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-undef, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /**
  * Allows for intuitive matrix input for matrix exercises.
  *

diff --git a/utils/mean-and-median.js b/utils/mean-and-median.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-redeclare, no-undef, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./stat.js");

diff --git a/utils/objective_.js b/utils/objective_.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-undef, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 /*
  * This file was copied from Perseus and shouldn't be modified directly.
  */

diff --git a/utils/parabola-intuition.js b/utils/parabola-intuition.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/polynomials.js b/utils/polynomials.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, one-var, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("./math.js");

diff --git a/utils/probability.js b/utils/probability.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/qhints.js b/utils/qhints.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable indent, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.fn["qhintsLoad"] = function() {

diff --git a/utils/rational-expressions.js b/utils/rational-expressions.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable camelcase, comma-dangle, eqeqeq, indent, max-len, no-redeclare, no-undef, no-unused-vars, prefer-template, space-before-blocks, space-infix-ops */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {

diff --git a/utils/scratchpad.js b/utils/scratchpad.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable camelcase, comma-dangle, indent, max-len, no-undef, no-unused-vars, one-var, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 require("../third_party/jquery.mobile.vmouse.js");

diff --git a/utils/slice-clone.js b/utils/slice-clone.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-undef, no-unused-vars, prefer-template */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 // Helper for fractions_cut_and_copy_1 and fractions_cut_and_copy_2

diff --git a/utils/spin.js b/utils/spin.js
@@ -1,3 +1,7 @@
+/* TODO(csilvers): fix these lint errors (http://eslint.org/docs/rules): */
+/* eslint-disable comma-dangle, indent, max-len, no-unused-vars */
+/* To fix, remove an entry above, run ka-lint, and fix errors. */
+
 define(function(require) {
 
 $.extend(KhanUtil, {