-
-
Save agustinhaller/5e489e5419e43b11d7b7 to your computer and use it in GitHub Desktop.
ionic Before Prepare Hooks
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#!/usr/bin/env node | |
var fs = require('fs'); | |
var path = require('path'); | |
var jshint = require('jshint').JSHINT; | |
var async = require('async'); | |
var foldersToProcess = [ | |
'js' | |
]; | |
foldersToProcess.forEach(function(folder) { | |
processFiles("www/" + folder); | |
}); | |
function processFiles(dir, callback) { | |
var errorCount = 0; | |
fs.readdir(dir, function(err, list) { | |
if (err) { | |
console.log('processFiles err: ' + err); | |
return; | |
} | |
async.eachSeries(list, function(file, innercallback) { | |
file = dir + '/' + file; | |
fs.stat(file, function(err, stat) { | |
if(!stat.isDirectory()) { | |
if(path.extname(file) === ".js") { | |
lintFile(file, function(hasError) { | |
if(hasError) { | |
errorCount++; | |
} | |
innercallback(); | |
}); | |
} else { | |
innercallback(); | |
} | |
} else { | |
innercallback(); | |
} | |
}); | |
}, function(error) { | |
if(errorCount > 0) { | |
process.exit(1); | |
} | |
}); | |
}); | |
} | |
function lintFile(file, callback) { | |
console.log("Linting " + file); | |
fs.readFile(file, function(err, data) { | |
if(err) { | |
console.log('Error: ' + err); | |
return; | |
} | |
if(jshint(data.toString())) { | |
console.log('File ' + file + ' has no errors.'); | |
console.log('-----------------------------------------'); | |
callback(false); | |
} else { | |
console.log('Errors in file ' + file); | |
var out = jshint.data(), | |
errors = out.errors; | |
for(var j = 0; j < errors.length; j++) { | |
console.log(errors[j].line + ':' + errors[j].character + ' -> ' + errors[j].reason + ' -> ' + | |
errors[j].evidence); | |
} | |
console.log('-----------------------------------------'); | |
callback(true); | |
} | |
}); | |
} |
Thanks for this script
I have modified with recursion, coloring, bug fixing
https://gist.github.com/49b36b4c7c3b5984d833.git
Hi Agustin,
Thanks for this. I ended up adding an array of files to ignore, i.e var filesToIgnore = [ 'angularfire.min.js', 'firebase.js', ... ]
and checking if the iterated file was one of them as these different files, which are not under my control, kept returning dozens of errors.
Thank you, @paulking00, that seems to work
Thanks Agustin. This script is really useful. I have modified this to implement recursion. https://gist.github.com/vinodpandey/06d4bcb080d51d3d148c36ba2410fc0a
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Great, thank you!
I have forked a version to add 1st level folder recursion
https://gist.github.com/paulking00/28f38ed875589c94f2f1