Removes custom test/reporter.js from the lint process
This commit is contained in:
parent
73e23bbc30
commit
1d066b768f
9
make.js
9
make.js
@ -1274,7 +1274,6 @@ target.lint = function() {
|
|||||||
'web/',
|
'web/',
|
||||||
'test/downloadutils.js',
|
'test/downloadutils.js',
|
||||||
'test/driver.js',
|
'test/driver.js',
|
||||||
'test/reporter.js',
|
|
||||||
'test/test.js',
|
'test/test.js',
|
||||||
'test/testutils.js',
|
'test/testutils.js',
|
||||||
'test/webbrowser.js',
|
'test/webbrowser.js',
|
||||||
@ -1292,10 +1291,10 @@ target.lint = function() {
|
|||||||
exec('npm install jshint@2.4.x'); // TODO read version from package.json
|
exec('npm install jshint@2.4.x'); // TODO read version from package.json
|
||||||
}
|
}
|
||||||
|
|
||||||
exit(exec('"' + jshintPath + '" --reporter test/reporter.js ' +
|
var exitCode = exec('"' + jshintPath + '" ' + LINT_FILES.join(' ')).code;
|
||||||
LINT_FILES.join(' ')).code);
|
if (exitCode === 0) {
|
||||||
|
echo('files checked, no errors found');
|
||||||
crlfchecker.checkIfCrlfIsPresent(LINT_FILES);
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
//
|
//
|
||||||
|
@ -1,47 +0,0 @@
|
|||||||
/* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
|
|
||||||
/* vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */
|
|
||||||
/* Copyright 2012 Mozilla Foundation
|
|
||||||
*
|
|
||||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
||||||
* you may not use this file except in compliance with the License.
|
|
||||||
* You may obtain a copy of the License at
|
|
||||||
*
|
|
||||||
* http://www.apache.org/licenses/LICENSE-2.0
|
|
||||||
*
|
|
||||||
* Unless required by applicable law or agreed to in writing, software
|
|
||||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
||||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
||||||
* See the License for the specific language governing permissions and
|
|
||||||
* limitations under the License.
|
|
||||||
*/
|
|
||||||
/* jshint node:true */
|
|
||||||
|
|
||||||
'use strict';
|
|
||||||
|
|
||||||
module.exports = {
|
|
||||||
reporter: function reporter(res) {
|
|
||||||
var len = 0;
|
|
||||||
var str = '';
|
|
||||||
|
|
||||||
res.forEach(function(r) {
|
|
||||||
var file = r.file;
|
|
||||||
var err = r.error;
|
|
||||||
|
|
||||||
switch (err.code) {
|
|
||||||
default:
|
|
||||||
len++;
|
|
||||||
str += file + ': line ' + err.line + ', col ' +
|
|
||||||
err.character + ', ' + err.reason + '\n';
|
|
||||||
}
|
|
||||||
});
|
|
||||||
|
|
||||||
if (str) {
|
|
||||||
process.stdout.write(str + '\n' + len + ' error' +
|
|
||||||
((len === 1) ? '' : 's') + '\n');
|
|
||||||
process.exit(2);
|
|
||||||
} else {
|
|
||||||
process.stdout.write('files checked, no errors found\n');
|
|
||||||
process.exit(0);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
};
|
|
Loading…
x
Reference in New Issue
Block a user