Show JSLint issues in Play20 browser error display instead of just in console

[sqs]

Here's a code snippet you can add to the README if you want:

  val localSettings = lintSettings ++ inConfig(Compile)(Seq(
    // jslint
    sourceDirectory in jslint <<= (baseDirectory)(_ / "app" / "assets" / "javascripts"),
    flags in jslint := Seq("anon", "sloppy", "nomen", "newcap"),
    indent in jslint := 2,
    predefs in jslint := List("angular", "Showdown", "moment", "_"),
    jslintConsoleOutput in Compile <<= (streams, formatter in (Compile, jslintConsoleOutput)) map { (s, f) =>
      (results: JSLintResults) => {
        results.filter(!_.getIssues.isEmpty).map(f.format).map(s.log.warn(_))
        results.filter(!_.getIssues.isEmpty).flatMap(_.getIssues.asScala).foreach { i =>
          throw new AssetCompilationException(Some(file(i.getSystemId)), "JSLint: " + i.getReason, Some(i.getLine),Some(i.getCharacter))
        }
      }
    }
  ))

[philcali]

That is excellent!

An FYI for other readers (there is nothing wrong with your solution), you could have also added that bit as an extra lint output, to segregate console output from other outputs. An example of that would have looked like:

// Streams here could be replaced by any TaskMap that would be relevant (like say project info?)
outputs in (Compile, jslint) <+= (streams) map { _ => (results) => {
  results.filter(!_.getIssues.isEmpty).flatMap(_.getIssues.asScala).foreach { i =>
    throw new AssetCompilationException(Some(file(i.getSystemId)), "JSLint: " + i.getReason, Some(i.getLine),Some(i.getCharacter))
  }}
}

Now, your console output is free to be silenced, or to be changed by the formatter, or something else without having to worry about your Play output being overridden.

[sqs]

Cool, thanks for the tip and for making sbt-jslint.

[sqs]

BTW, I had to add a type annotation to (results) on the first line for this to compile (sbt 0.12.1).

// Streams here could be replaced by any TaskMap that would be relevant (like say project info?)
outputs in (Compile, jslint) <+= (streams) map { _ => (results: JSLintResults) => {
  results.filter(!_.getIssues.isEmpty).flatMap(_.getIssues.asScala).foreach { i =>
    throw new AssetCompilationException(Some(file(i.getSystemId)), "JSLint: " + i.getReason, Some(i.getLine),Some(i.getCharacter))
  }}
}