Mobile/Fennec/Android/Static analysis/checkstyle-idea: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Replace link to checkstyle.xml with link to raw version (previous link goes to html mxr page, which the plugin can't handle))
 
Line 17: Line 17:
* Click the plus at the bottom of the "Configuration file" box
* Click the plus at the bottom of the "Configuration file" box
* Add a description (e.g. "mozilla")
* Add a description (e.g. "mozilla")
* Add the path to the checkstyle configuration: <pre>http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/checkstyle.xml</pre>
* Add the path to the checkstyle configuration: <pre>http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/checkstyle.xml?raw=1</pre>
* Accept
* Accept
* Click the "Active" checkbox next to the new configuration
* Click the "Active" checkbox next to the new configuration
* Hit "OK" to close the preferences
* Hit "OK" to close the preferences

Latest revision as of 16:41, 2 May 2016

You can run checkstyle via the checkstyle-idea plugin within Android Studio or Intellij. It will highlight violations in red, e.g. package statements should not span more than one line:

Example of checkstyle-idea failure

Installation

  • Open the preferences
  • Find "Plugins" in the preferences
  • Click "Browse repositories"
  • Search for "checkstyle".
  • Select CheckStyle-IDEA
  • Click "Install plugin" on the right & confirm
  • Click restart Android Studio

Adding the config file

  • Open the preferences
  • Go to (or search for) "Checkstyle" menu item (it's under "Other Settings")
  • Click the plus at the bottom of the "Configuration file" box
  • Add a description (e.g. "mozilla")
  • Add the path to the checkstyle configuration:
    http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/checkstyle.xml?raw=1
  • Accept
  • Click the "Active" checkbox next to the new configuration
  • Hit "OK" to close the preferences