Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 442bd8b

Browse files
committedFeb 1, 2013
Issue mozilla#2654 - Lint with Travis-CI
1 parent 43d6c6f commit 442bd8b

File tree

4 files changed

+21
-1
lines changed

4 files changed

+21
-1
lines changed
 

‎.travis.yml

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
language: node_js
2+
node_js:
3+
- 0.8

‎make.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -994,7 +994,7 @@ target.jshint = function() {
994994
//'extensions/chrome/*.js'
995995
];
996996

997-
exec('jshint --reporter test/reporter.js ' + LINT_FILES.join(' '));
997+
exit(exec('./node_modules/.bin/jshint --reporter test/reporter.js ' + LINT_FILES.join(' ')).code);
998998
};
999999

10001000
//

‎package.json

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
{
2+
"name": "pdf.js",
3+
"version": "0.7.169",
4+
"dependencies": {
5+
"jshint": "git://github.com/jshint/jshint.git#42ace75a"
6+
},
7+
"scripts": {
8+
"test": "node make jshint"
9+
},
10+
"repository": {
11+
"type": "git",
12+
"url": "git://github.com/mozilla/pdf.js.git"
13+
}
14+
}

‎test/reporter.js

+3
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@ module.exports = {
2323
if (str) {
2424
process.stdout.write(str + "\n" + len + " error" +
2525
((len === 1) ? "" : "s") + "\n");
26+
process.exit(2);
27+
} else {
28+
process.exit(0);
2629
}
2730
}
2831
};

0 commit comments

Comments
 (0)
Please sign in to comment.