Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Updated return codes from jshint #156

Merged
merged 1 commit into from

2 participants

@camilonova

jshint return codes changed after version 1.0, more information here:

jshint/jshint#916 (comment)

@camilonova camilonova Updated return codes from jshint
jshint return codes changed after version 1.0, more information here:

jshint/jshint#916 (comment)
5e71cc9
@kmmbvnr
Owner

Thanks

@kmmbvnr kmmbvnr closed this
@kmmbvnr kmmbvnr reopened this
@kmmbvnr kmmbvnr merged commit af6953c into kmmbvnr:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 13, 2013
  1. @camilonova

    Updated return codes from jshint

    camilonova authored
    jshint return codes changed after version 1.0, more information here:
    
    jshint/jshint#916 (comment)
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  django_jenkins/tasks/run_jshint.py
View
2  django_jenkins/tasks/run_jshint.py
@@ -53,7 +53,7 @@ def teardown_test_environment(self, **kwargs):
process = subprocess.Popen(cmd, stdout=subprocess.PIPE)
output, err = process.communicate()
retcode = process.poll()
- if retcode not in [0, 1]: # normal csslint return codes
+ if retcode not in [0, 2]: # normal jshint return codes
raise CalledProcessError(retcode, cmd, output=output + '\n' + err)
self.output.write(output.decode('utf-8'))
Something went wrong with that request. Please try again.