Improve JS-related config

Summary: No more pretending we would be running JS tests. No more jshint against code we don't care about. Also makes `make lint` do something useful. Test Plan: Manual Reviewers: paulruan Reviewed By: paulruan Subscribers: changesbot Differential Revision: https://tails.corp.dropbox.com/D200054
dropbox · Aug 15, 2016 · b17aaa0 · b17aaa0
1 parent abd7838
commit b17aaa0
Show file tree

Hide file tree

Showing 374 changed files with 73 additions and 67,125 deletions.
diff --git a/Makefile b/Makefile
@@ -82,8 +82,6 @@ lint-js:
 test-full: install-test-requirements
 	$(MAKE) lint
 	$(MAKE) coverage
-	@echo SKIPPING JAVASCRIPT TESTS UNTIL WE FIX THE RUNNER.
-	#@npm run test-ci
 
 coverage:
 	coverage run -m py.test --junitxml=python.junit.xml tests

diff --git a/node_modules/jshint/README.md b/node_modules/jshint/README.md
diff --git a/node_modules/jshint/bin/apply b/node_modules/jshint/bin/apply
diff --git a/node_modules/jshint/bin/build b/node_modules/jshint/bin/build
diff --git a/node_modules/jshint/bin/changelog b/node_modules/jshint/bin/changelog
diff --git a/node_modules/jshint/bin/jshint b/node_modules/jshint/bin/jshint
diff --git a/node_modules/jshint/bin/land b/node_modules/jshint/bin/land
diff --git a/node_modules/jshint/data/ascii-identifier-data.js b/node_modules/jshint/data/ascii-identifier-data.js
diff --git a/node_modules/jshint/data/non-ascii-identifier-part-only.js b/node_modules/jshint/data/non-ascii-identifier-part-only.js
diff --git a/node_modules/jshint/data/non-ascii-identifier-start.js b/node_modules/jshint/data/non-ascii-identifier-start.js