diff --git a/bin/sanitycheck.js b/bin/sanitycheck.js index 3bb9a822a..041540606 100755 --- a/bin/sanitycheck.js +++ b/bin/sanitycheck.js @@ -19,6 +19,7 @@ try { var BASEDIR = __dirname+"/../"; var APPSDIR_RELATIVE = "apps/"; var APPSDIR = BASEDIR + APPSDIR_RELATIVE; +var knownWarningCount = 0; var warningCount = 0; var errorCount = 0; function ERROR(msg, opt) { @@ -32,10 +33,11 @@ function WARN(msg, opt) { opt = opt||{}; if (KNOWN_WARNINGS.includes(msg)) { console.log(`Known warning : ${msg}`); + knownWarningCount++; } else { console.log(`::warning${Object.keys(opt).length?" ":""}${Object.keys(opt).map(k=>k+"="+opt[k]).join(",")}::${msg}`); + warningCount++; } - warningCount++; } var apps = []; @@ -396,8 +398,11 @@ while(fileA=allFiles.pop()) { } console.log("=================================="); -console.log(`${errorCount} errors, ${warningCount} warnings`); +console.log(`${errorCount} errors, ${warningCount} warnings (and ${knownWarningCount} known warnings)`); console.log("=================================="); if (errorCount) { process.exit(1); +} else if ("CI" in process.env && warningCount) { + console.log("Running in CI, raising an error from warnings"); + process.exit(1); }