Page MenuHomePhabricator

Add Scalastyle linter
Needs ReviewPublic

Authored by onur.cakmak on Sep 23 2015, 5:09 PM.
Tags
None
Referenced Files
F15474056: D14145.diff
Sun, Apr 6, 4:59 AM
F15462845: D14145.id34197.diff
Tue, Apr 1, 6:06 PM
F15462815: D14145.id34179.diff
Tue, Apr 1, 5:55 PM
F15462478: D14145.id34179.diff
Tue, Apr 1, 3:19 PM
F15461186: D14145.id.diff
Tue, Apr 1, 6:00 AM
F15460026: D14145.diff
Mon, Mar 31, 7:26 PM
F15459303: D14145.diff
Mon, Mar 31, 11:49 AM
F15448494: D14145.diff
Fri, Mar 28, 5:14 AM

Details

Summary

A scalastyle linter that we have been testing internally for some time.
The implementation doesn't depend on sbt or a wrapper script.

Test Plan

Add a scalastyle linter with a configuration pointing to the scalastyle files and break a scalastyle rule.

Example configuration:

.arclint
{
  "linters": {
    "scalastyle": {
      "type": "scalastyle",
      "scalastyle.config": "support/scalastyle/scalastyle-config.xml",
      "scalastyle.jar": "support/scalastyle/scalastyle_2.11-0.6.0-batch.jar",
      "include": "(\\.scala$)"
    }
  }
}

Diff Detail

Repository
rARC Arcanist
Branch
feature-scalastyle
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8029
Build 9088: [Placeholder Plan] Wait for 30 Seconds
Build 9087: arc lint + arc unit