Ecosyste.ms: Awesome
An open API service indexing awesome lists of open source software.
https://github.com/scapegoat-scala/scapegoat
Scala compiler plugin for static code analysis
https://github.com/scapegoat-scala/scapegoat
hacktoberfest inspection scala scala-compiler
Last synced: 6 days ago
JSON representation
Scala compiler plugin for static code analysis
- Host: GitHub
- URL: https://github.com/scapegoat-scala/scapegoat
- Owner: scapegoat-scala
- License: apache-2.0
- Created: 2013-11-21T17:45:58.000Z (about 11 years ago)
- Default Branch: master
- Last Pushed: 2024-12-30T12:53:13.000Z (12 days ago)
- Last Synced: 2025-01-02T10:58:44.100Z (10 days ago)
- Topics: hacktoberfest, inspection, scala, scala-compiler
- Language: Scala
- Homepage:
- Size: 2.25 MB
- Stars: 534
- Watchers: 20
- Forks: 92
- Open Issues: 53
-
Metadata Files:
- Readme: README.md
- License: LICENSE
Awesome Lists containing this project
- awesome-linters - scapegoat - Another similar tool to (Linters / Scala)
README
Scapegoat
=========
[](https://codecov.io/gh/sksamuel/scapegoat)
[
](http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22scalac-scapegoat-plugin_2.12.19%22)
[
](http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22scalac-scapegoat-plugin_2.13.14%22)
[
](http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22scalac-scapegoat-plugin_3.3.3%22)
[
](http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22scalac-scapegoat-plugin_3.4.2%22)
[](https://scala-steward.org)
Scapegoat is a Scala static code analyzer, which is more colloquially known as a code lint tool or linter. Scapegoat works in a similar vein to Java's [FindBugs](http://findbugs.sourceforge.net/) or [checkstyle](http://checkstyle.sourceforge.net/), or Scala's [Scalastyle](https://github.com/scalastyle/scalastyle).
A static code analyzer is a tool that flags suspicious language usage in code. This can include behavior likely to lead to bugs, non-idiomatic usage of a language, or just code that doesn't conform to specified style guidelines.
**What's the difference between this project and Scalastyle (or others)?**
Scalastyle is a similar linting tool which focuses mostly on enforcing style/code standards. There are no problems in running multiple analysis tools on the same codebase. In fact, it could be beneficial as the total set of possible warnings is the union of the inspections of all the enabled tools. The worst case is that the same warnings might get generated by multiple tools.
## Usage
Scapegoat is developed as a scala compiler plugin, which can then be used inside your build tool.
### SBT
See: [sbt-scapegoat](https://github.com/sksamuel/sbt-scapegoat) for SBT integration.
### Maven
Firstly you need to add scapegoat plugin as a dependency:
```xml
com.sksamuel.scapegoat
scalac-scapegoat-plugin_${scala.version}
1.3.12
```
Then configure `scala-maven-plugin` by adding `compilerPlugin`
```xml
net.alchim31.maven
scala-maven-plugin
-P:scapegoat:dataDir:./target/scapegoat
com.sksamuel.scapegoat
scalac-scapegoat-plugin_${scala.binary.version}
1.4.6
```
The only required parameter is `dataDir` (where report will be generated):
`-P:scapegoat:dataDir:./target/scapegoat`
You can pass other configuration flags same way, e.g.
`-P:scapegoat:disabledInspections:FinalModifierOnCaseClass`
Note: You may use a separate maven profile, so that the dependency doesn't go to you runtime classpath.
### Gradle with a plugin
Use [gradle-scapegoat-plugin by @eugene-sy](https://github.com/eugene-sy/gradle-scapegoat-plugin)
### Gradle - manually
Firstly you need to add scapegoat plugin as a dependency:
```groovy
dependencies {
compile 'com.sksamuel.scapegoat:scalac-scapegoat-plugin_2.12.14:1.4.6'
scalaCompilerPlugin "com.sksamuel.scapegoat:scalac-scapegoat-plugin_2.12.14:1.4.6"
}
```
Then configure `scala-compiler-plugin`
```groovy
configurations {
scalaCompilerPlugin
}
tasks.withType(ScalaCompile) {
scalaCompileOptions.additionalParameters = [
"-Xplugin:" + configurations.scalaCompilerPlugin.asPath,
"-P:scapegoat:dataDir:" + buildDir + "/scapegoat"
]
}
```
The only required parameter is `dataDir` (where report will be generated):
`"-P:scapegoat:dataDir:" + buildDir + "/scapegoat",`
You can pass other configuration flags adding it to the `additionalParameters` list, e.g.
`"-P:scapegoat:disabledInspections:FinalModifierOnCaseClass"`
## Full list of compiler flags
| Flag | Parameters | Required |
|-------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------|----------|
| `-P:scapegoat:dataDir:` | Path to reports directory for the plugin. | true |
| `-P:scapegoat:disabledInspections:` | Colon separated list of disabled inspections (defaults to none). | false |
| `-P:scapegoat:enabledInspections:` | Colon separated list of enabled inspections (defaults to all). | false |
| `-P:scapegoat:customInspectors:` | Colon separated list of custom inspections. | false |
| `-P:scapegoat:ignoredFiles:` | Colon separated list of regexes to match files to ignore. | false |
| `-P:scapegoat:verbose:` | Boolean flag that enables/disables verbose console messages. | false |
| `-P:scapegoat:consoleOutput:` | Boolean flag that enables/disables console report output. | false |
| `-P:scapegoat:reports:` | Colon separated list of reports to generate. Valid options are `none`, `xml`, `html`, `scalastyle`, `markdown`, ,`gitlab-codequality`, or `all`. | false |
| `-P:scapegoat:overrideLevels:` | Overrides the built in warning levels. Should be a colon separated list of `name=level` expressions. | false |
| `-P:scapegoat:sourcePrefix:` | Overrides source prefix if it differs from `src/main/scala`, for ex. `app/` for Play applications. | false |
| `-P:scapegoat:minimalLevel:` | Provides minimal level of inspection displayed in reports and in the console. | false |
## Reports
Here is sample output from the console during the build for a project with warnings/errors:
```
[warning] [scapegoat] Unused method parameter - org.ensime.util.ClassIterator.scala:46
[warning] [scapegoat] Unused method parameter - org.ensime.util.ClassIterator.scala:137
[warning] [scapegoat] Use of var - org.ensime.util.ClassIterator.scala:22
[warning] [scapegoat] Use of var - org.ensime.util.ClassIterator.scala:157
[ info] [scapegoat]: Inspecting compilation unit [FileUtil.scala]
[warning] [scapegoat] Empty if statement - org.ensime.util.FileUtil.scala:157
[warning] [scapegoat] Expression as statement - org.ensime.util.FileUtil.scala:180
```
And if you prefer a prettier report, here is a screen shot of the type of HTML report scapegoat generates:
## Configuration
For instructions on suppressing warnings by file, by inspection or by line see [the sbt-scapegoat README](https://github.com/sksamuel/sbt-scapegoat).
To suppress warnings globally for the project, use `disabledInspections` or `overrideLevels` flags:
```
-P:scapegoat:disabledInspections:FinalModifierOnCaseClass
-P:scapegoat:overrideLevels:PreferSeqEmpty=ignore:AsInstanceOf=ignore
```
### Inspections
There are currently 123 inspections for Scala 2, and 3 for Scala 3.
An overview list is given, followed by a more detailed description of each inspection after the list (todo: finish rest of detailed descriptions)
| Name | Brief Description | Default Level | Scala 2 | Scala 3 |
|---------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------|---------------|---------|---------|
| AbstractTrait | Check if trait is abstract | Info | Yes | Yes |
| ArrayEquals | Checks for comparison of arrays using `==` which will always return false | Info | Yes | No |
| ArraysInFormat | Checks for arrays passed to String.format | Error | Yes | No |
| ArraysToString | Checks for explicit toString calls on arrays | Warning | Yes | No |
| AsInstanceOf | Checks for use of `asInstanceOf` | Warning | Yes | No |
| AvoidOperatorOverload | Checks for mental symbolic method names | Info | Yes | No |
| AvoidRequire | Use of require | Warning | Yes | Yes |
| AvoidSizeEqualsZero | Traversable.size can be slow for some data structure, prefer .isEmpty | Warning | Yes | No |
| AvoidSizeNotEqualsZero | Traversable.size can be slow for some data structure, prefer .nonEmpty | Warning | Yes | No |
| AvoidToMinusOne | Checks for loops that use `x to n-1` instead of `x until n` | Info | Yes | No |
| BigDecimalDoubleConstructor | Checks for use of `BigDecimal(double)` which can be unsafe | Warning | Yes | No |
| BigDecimalScaleWithoutRoundingMode | `setScale()` on a `BigDecimal` without setting the rounding mode can throw an exception | Warning | Yes | No |
| BooleanParameter | Checks for functions that have a Boolean parameter | Info | Yes | No |
| BoundedByFinalType | Looks for types with upper bounds of a final type | Warning | Yes | No |
| BrokenOddness | Checks for a % 2 == 1 for oddness because this fails on negative numbers | Warning | Yes | No |
| CatchException | Checks for try blocks that catch Exception | Warning | Yes | No |
| CatchExceptionImmediatelyRethrown | Checks for try-catch blocks that immediately rethrow caught exceptions. | Warning | Yes | No |
| CatchFatal | Checks for try blocks that catch fatal exceptions: VirtualMachineError, ThreadDeath, InterruptedException, LinkageError, ControlThrowable | Warning | Yes | No |
| CatchNpe | Checks for try blocks that catch null pointer exceptions | Error | Yes | No |
| CatchThrowable | Checks for try blocks that catch Throwable | Warning | Yes | No |
| ClassNames | Ensures class names adhere to the style guidelines | Info | Yes | No |
| CollectionIndexOnNonIndexedSeq | Checks for indexing on a Seq which is not an IndexedSeq | Warning | Yes | No |
| CollectionNamingConfusion | Checks for variables that are confusingly named | Info | Yes | No |
| CollectionNegativeIndex | Checks for negative access on a sequence eg `list.get(-1)` | Warning | Yes | No |
| CollectionPromotionToAny | Checks for collection operations that promote the collection to `Any` | Warning | Yes | No |
| ComparingFloatingPointTypes | Checks for equality checks on floating point types | Error | Yes | No |
| ComparingUnrelatedTypes | Checks for equality comparisons that cannot succeed | Error | Yes | No |
| ComparisonToEmptyList | Checks for code like `a == List()` or `a == Nil` | Info | Yes | No |
| ComparisonToEmptySet | Checks for code like `a == Set()` or `a == Set.empty` | Info | Yes | No |
| ComparisonWithSelf | Checks for equality checks with itself | Warning | Yes | No |
| ConstantIf | Checks for code where the if condition compiles to a constant | Warning | Yes | No |
| DivideByOne | Checks for divide by one, which always returns the original value | Warning | Yes | No |
| DoubleNegation | Checks for code like `!(!b)` | Info | Yes | No |
| DuplicateImport | Checks for import statements that import the same selector | Info | Yes | No |
| DuplicateMapKey | Checks for duplicate key names in Map literals | Warning | Yes | No |
| DuplicateSetValue | Checks for duplicate values in set literals | Warning | Yes | No |
| EitherGet | Checks for use of .get on Left or Right | Error | Yes | No |
| EmptyCaseClass | Checks for case classes like `case class Faceman()` | Info | Yes | No |
| EmptyFor | Checks for empty `for` loops | Warning | Yes | No |
| EmptyIfBlock | Checks for empty `if` blocks | Warning | Yes | No |
| EmptyInterpolatedString | Looks for interpolated strings that have no arguments | Error | Yes | No |
| EmptyMethod | Looks for empty methods | Warning | Yes | No |
| EmptySynchronizedBlock | Looks for empty synchronized blocks | Warning | Yes | No |
| EmptyTryBlock | Looks for empty try blocks | Warning | Yes | No |
| EmptyWhileBlock | Looks for empty while loops | Warning | Yes | No |
| ExistsSimplifiableToContains | `exists(x => x == b)` replaceable with `contains(b)` | Info | Yes | No |
| FilterDotHead | `.filter(x => ).head` can be replaced with `find(x => ) match { .. } ` | Info | Yes | No |
| FilterDotHeadOption | `.filter(x =>).headOption` can be replaced with `find(x => )` | Info | Yes | No |
| FilterDotIsEmpty | `.filter(x => ).isEmpty` can be replaced with `!exists(x => )` | Info | Yes | No |
| FilterDotSize | `.filter(x => ).size` can be replaced more concisely with with `count(x => )` | Info | Yes | No |
| FilterOptionAndGet | `.filter(_.isDefined).map(_.get)` can be replaced with `flatten` | Info | Yes | No |
| FinalModifierOnCaseClass | Using Case classes without `final` modifier can lead to surprising breakage | Info | Yes | No |
| FinalizerWithoutSuper | Checks for overridden finalizers that do not call super | Warning | Yes | No |
| FindAndNotEqualsNoneReplaceWithExists | `.find(x => ) != None` can be replaced with `exist(x => )` | Info | Yes | No |
| FindDotIsDefined | `find(x => ).isDefined` can be replaced with `exist(x => )` | Info | Yes | No |
| IllegalFormatString | Looks for invalid format strings | Error | Yes | No |
| ImpossibleOptionSizeCondition | Checks for code like `option.size > 2` which can never be true | Error | Yes | No |
| IncorrectNumberOfArgsToFormat | Checks for wrong number of arguments to `String.format` | Error | Yes | No |
| IncorrectlyNamedExceptions | Checks for exceptions that are not called *Exception and vice versa | Error | Yes | No |
| InvalidRegex | Checks for invalid regex literals | Info | Yes | No |
| IsInstanceOf | Checks for use of `isInstanceOf` | Warning | Yes | No |
| JavaConversionsUse | Checks for use of implicit Java conversions | Warning | Yes | No |
| ListAppend | Checks for List :+ which is O(n) | Info | Yes | No |
| ListSize | Checks for `List.size` which is O(n). | Info | Yes | No |
| LonelySealedTrait | Checks for sealed traits which have no implementation | Error | Yes | No |
| LooksLikeInterpolatedString | Finds strings that look like they should be interpolated but are not | Warning | Yes | No |
| MapGetAndGetOrElse | `Map.get(key).getOrElse(value)` can be replaced with `Map.getOrElse(key, value)` | Error | Yes | No |
| MaxParameters | Checks for methods that have over 10 parameters | Info | Yes | No |
| MethodNames | Warns on method names that don't adhere to the Scala style guidelines | Info | Yes | No |
| MethodReturningAny | Checks for defs that are defined or inferred to return `Any` | Warning | Yes | No |
| ModOne | Checks for `x % 1` which will always return `0` | Warning | Yes | No |
| NanComparison | Checks for `x == Double.NaN` which will always fail | Error | Yes | No |
| NegationIsEmpty | `!Traversable.isEmpty` can be replaced with `Traversable.nonEmpty` | Info | Yes | No |
| NegationNonEmpty | `!Traversable.nonEmpty` can be replaced with `Traversable.isEmpty` | Info | Yes | No |
| NoOpOverride | Checks for code that overrides parent method but simply calls super | Info | Yes | No |
| NullAssignment | Checks for use of `null` in assignments | Warning | Yes | No |
| NullParameter | Checks for use of `null` in method invocation | Warning | Yes | No |
| ObjectNames | Ensures object names adhere to the Scala style guidelines | Info | Yes | No |
| OptionGet | Checks for `Option.get` | Error | Yes | Yes |
| OptionSize | Checks for `Option.size` | Error | Yes | No |
| ParameterlessMethodReturnsUnit | Checks for `def foo : Unit` | Warning | Yes | No |
| PartialFunctionInsteadOfMatch | Warns when you could use a partial function directly instead of a match block | Info | Yes | No |
| PointlessTypeBounds | Finds type bounds of the form `[A <: Any]` or `[A >: Nothing]` | Warning | Yes | No |
| PreferMapEmpty | Checks for Map() when could use Map.empty | Info | Yes | No |
| PreferSeqEmpty | Checks for Seq() when could use Seq.empty | Info | Yes | No |
| PreferSetEmpty | Checks for Set() when could use Set.empty | Info | Yes | No |
| ProductWithSerializableInferred | Checks for vals that have `Product with Serializable` as their inferred type | Warning | Yes | No |
| PublicFinalizer | Checks for overridden finalizes that are public | Info | Yes | No |
| RedundantFinalModifierOnMethod | Redundant `final` modifier on method that cannot be overridden | Info | Yes | No |
| RedundantFinalModifierOnVar | Redundant `final` modifier on var that cannot be overridden | Info | Yes | No |
| RedundantFinalizer | Checks for empty finalizers. | Warning | Yes | No |
| RepeatedCaseBody | Checks for case statements which have the same body | Warning | Yes | No |
| RepeatedIfElseBody | Checks for the main branch and the else branch of an `if` being the same | Warning | Yes | No |
| ReverseFunc | `reverse` followed by `head`, `headOption`, `iterator`, or`map` can be replaced, respectively, with `last`, `lastOption`, `reverseIterator`, or `reverseMap` | Info | Yes | No |
| ReverseTailReverse | `.reverse.tail.reverse` can be replaced with `init` | Info | Yes | No |
| ReverseTakeReverse | `.reverse.take(...).reverse` can be replaced with `takeRight` | Info | Yes | No |
| SimplifyBooleanExpression | `b == false` can be simplified to `!b` | Info | Yes | No |
| StoreBeforeReturn | Checks for storing a value in a block, and immediately returning the value | Info | Yes | No |
| StripMarginOnRegex | Checks for .stripMargin on regex strings that contain a pipe | Error | Yes | No |
| SubstringZero | Checks for `String.substring(0)` | Info | Yes | No |
| SuspiciousMatchOnClassObject | Finds code where matching is taking place on class literals | Warning | Yes | No |
| SwallowedException | Finds catch blocks that don't handle caught exceptions | Warning | Yes | No |
| SwapSortFilter | `sort.filter` can be replaced with `filter.sort` for performance | Info | Yes | No |
| TryGet | Checks for use of `Try.get` | Error | Yes | No |
| TypeShadowing | Checks for shadowed type parameters in methods | Warning | Yes | No |
| UnnecessaryConversion | Checks for unnecessary `toInt` on instances of Int or `toString` on Strings, etc. | Warning | Yes | No |
| UnnecessaryIf | Checks for code like `if (expr) true else false` | Info | Yes | No |
| UnnecessaryReturnUse | Checks for use of `return` keyword in blocks | Info | Yes | No |
| UnreachableCatch | Checks for catch clauses that cannot be reached | Warning | Yes | No |
| UnsafeContains | Checks for `List.contains(value)` for invalid types | Error | Yes | No |
| UnsafeStringContains | Checks for `String.contains(value)` for invalid types | Error | Yes | No |
| UnsafeTraversableMethods | Check unsafe traversable method usages (head, tail, init, last, reduce, reduceLeft, reduceRight, max, maxBy, min, minBy) | Error | Yes | No |
| UnusedMethodParameter | Checks for unused method parameters | Warning | Yes | No |
| UseCbrt | Checks for use of `math.pow` for calculating `math.cbrt` | Info | Yes | No |
| UseExpM1 | Checks for use of `math.exp(x) - 1` instead of `math.expm1(x)` | Info | Yes | No |
| UseLog10 | Checks for use of `math.log(x)/math.log(10)` instead of `math.log10(x)` | Info | Yes | No |
| UseLog1P | Checks for use of `math.log(x + 1)` instead of `math.log1p(x)` | Info | Yes | No |
| UseSqrt | Checks for use of `math.pow` for calculating `math.sqrt` | Info | Yes | No |
| VarClosure | Finds closures that reference var | Warning | Yes | No |
| VarCouldBeVal | Checks for `var`s that could be declared as `val`s | Warning | Yes | No |
| VariableShadowing | Checks for multiple uses of the variable name in nested scopes | Warning | Yes | No |
| WhileTrue | Checks for code that uses a `while(true)` or `do { } while(true)` block. | Warning | Yes | No |
| ZeroNumerator | Checks for dividing by 0 by a number, eg `0 / x` which will always return `0` | Warning | Yes | No |
##### Arrays to string
Checks for explicit toString calls on arrays. Since toString on an array does not perform a deep toString, like say scala's List, this is usually a mistake.
##### CollectionIndexOnNonIndexedSeq
Checks for calls of `.apply(idx)` on a `Seq` where the index is not a literal and the `Seq` is not an `IndexedSeq`.
*Rationale* If code which expects O(1) positional access to a Seq is given a non-IndexedSeq (such as a List, where indexing is O(n)) then this may cause poor performance.
##### ComparingUnrelatedTypes
Checks for equality comparisons that cannot succeed because the types are unrelated. Eg `"string" == BigDecimal(1.0)`. The scala compiler has a less strict version of this inspection.
##### ConstantIf
Checks for if statements where the condition is always true or false. Not only checks for the boolean literals, but also any expression that the compiler is able to turn into a constant value. Eg, `if (0 < 1) then else that`
##### IllegalFormatString
Checks for a format string that is not invalid, such as invalid conversions, invalid flags, etc. Eg, `"% s"`, `"%qs"`, `%.-4f"`
##### IncorrectNumberOfArgsToFormat
Checks for an incorrect number of arguments to String.format. Eg, `"%s %s %f".format("need", "three")` flags an error because the format string specifies 3 parameters but the call only provides 2.
##### InvalidRegex
Checks for invalid regex literals that would fail at compile time. Either dangling metacharacters, or unclosed escape characters, etc that kind of thing.
##### List size
Checks for .size on an instance of List. Eg, `val a = List(1,2,3); a.size`
*Rationale*: List.size is O(n) so for performance reasons if .size is needed on a list that could be large, consider using an alternative with O(1), eg Array, Vector or ListBuffer.
##### Redundant finalizer
Checks for empty finalizers. This is redundant code and should be removed. Eg, `override def finalize : Unit = { }`
##### PreferSetEmpty
Indicates where code using Set() could be replaced with Set.empty. Set() instantiates a new instance each time it is invoked, whereas Set.empty returns a pre-instantiated instance.
##### UnnecessaryReturnUse
Checks for use of return in a function or method. Since the final expression of a block is always the return value, using return is unnecessary. Eg, `def foo = { println("hello"); return 12; }`
##### UnreachableCatch
Checks for catch clauses that cannot be reached. This means the exception is dead and if you want that exception to take precedence you should move up further up the case list.
##### UnsafeContains
Checks for `List.contains(value)` for invalid types. The method for contains accepts any types. This inspection finds situations when you have a list of type A and you are checking for contains on type B which cannot hold.
##### While true
Checks for code that uses a `while(true)` or `do { } while(true)` block.
*Rationale*: This type of code is usually not meant for production as it will not return normally. If you need to loop until interrupted, then consider using a flag.
### Suppressing Warnings by Method or Class
You can suppress a specific warning by method or by class using the java.lang.SuppressWarnings annotation.
Use the simple name of the inspection to be ignored as the argument, or use "all" to suppress all scapegoat warnings in the specified scope.
Some examples:
```scala
@SuppressWarnings(Array("all"))
class Test {
def hello: Unit = {
val s: Any = "sammy"
println(s.asInstanceOf[String])
}
}
class Test2 {
@SuppressWarnings(Array("AsInstanceOf"))
def hello: Unit = {
val s: Any = "sammy"
println(s.asInstanceOf[String])
}
}
```
## Other static analysis tools:
* ScalaStyle (Scala) - https://github.com/scalastyle/scalastyle/wiki
* Linter (Scala) - https://github.com/HairyFotr/linter
* WartRemover (Scala) - https://github.com/puffnfresh/wartremover
* Findbugs (JVM) - http://findbugs.sourceforge.net/bugDescriptions.html
* Fb-contrib (JVM) - http://fb-contrib.sourceforge.net/
* CheckStyle (Java) - http://checkstyle.sourceforge.net/availablechecks.html
* PMD (Java) - http://pmd.sourceforge.net/pmd-5.0.3/rules/index.html
* Error-prone (Java) - https://github.com/google/error-prone
* CodeNarc (Groovy) - http://codenarc.sourceforge.net/codenarc-rule-index.html
* PVS-Studio (C++) - http://www.viva64.com/en/d/
* Coverity (C++) - http://www.slideshare.net/Coverity/static-analysis-primer-22874326 (6,7)
* CppCheck (C++) - http://cppcheck.sourceforge.net/
* OCLint (C++/ObjC) - http://docs.oclint.org/en/dev/rules/index.html
* JSHint (Javascript) - http://jshint.com/
* JavascriptLint (Javascript) - http://www.javascriptlint.com/
* ClosureLinter (Javascript) - https://developers.google.com/closure/utilities/