-
-
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); | |
} | |
}); | |
} |
Great, thank you!
I have forked a version to add 1st level folder recursion
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
Rather than putting all my code into the
js
folder, I've instead set-up a folder structure inside anapp
folder, which contains subfolders with additional javascript in them. How can I change this to search in subfolders too?I've changed line 9 to
'app'
, and that picks up theapp.js
file inside it, bit nothing in any of the subfolders.Cheers.