Merge pull request #10622 from edx/clytwynec/jshint

Clytwynec/jshint
This commit is contained in:
Christine Lytwynec
2015-11-16 08:59:41 -05:00
3 changed files with 5 additions and 7 deletions

View File

@@ -1,2 +1,2 @@
common/static/js/vendor
lms/static/js/vendor
**/vendor
node_modules

View File

@@ -272,11 +272,9 @@ def run_jshint(options):
jshint_report = jshint_report_dir / "jshint.report"
_prepare_report_dir(jshint_report_dir)
jshint_directories = ["common/static/js", "cms/static/js", "lms/static/js"]
sh(
"jshint {list} --config .jshintrc >> {jshint_report}".format(
list=(" ".join(jshint_directories)), jshint_report=jshint_report
"jshint {root} --config .jshintrc >> {jshint_report}".format(
root=Env.REPO_ROOT, jshint_report=jshint_report
),
ignore_error=True
)

View File

@@ -12,7 +12,7 @@ set -e
# Violations thresholds for failing the build
export PYLINT_THRESHOLD=5650
export JSHINT_THRESHOLD=3700
export JSHINT_THRESHOLD=9390
doCheckVars() {
if [ -n "$CIRCLECI" ] ; then