Mobile/Fennec/Android/Static analysis: Difference between revisions

Revision as of 18:11, 25 March 2016

Catches various errors & optimizations specifically related to Android code and resources

./mach gradle app:lint

example checks? `lint --show`
in automation? Treeherder tier 2 "Unit" job
config? mobile/android/app/lint.xml
open issues? Run locally for a complete list.
meta bug? meta bug
If you fix all issues for a warning, modify lint.xml to change your warning into an error!

Currently detects NullPointerExceptions and resource leaks in Android applications.

It's currently unknown how to run locally.

Checks code style

./mach gradle app:checkstyle

Checks for Javascript errors (e.g. syntax errors, almost like a compiled language! :D)

./mach eslint --setup  # run once, or if the command breaks for some reason
./mach eslint mobile/android

example checks? http://eslint.org/docs/rules/
in automation? Treeherder tier 1 "ES" job
config? mobile/*.eslintrc & root .eslintignore
open issues? Open a .eslintrc and enable checks.
meta bug? meta bug
If you fix all issues for a warning, modify the appropriate .eslintrc to change your warning into an error!

@@ Line 4: / Line 4: @@
 * '''example checks?''' [https://sites.google.com/a/android.com/tools/tips/lint-checks `lint --show`]
-* '''in automation?''' TBPL tier 2 "unit" job
+* '''in automation?''' Treeherder tier 2 "Unit" job
 * '''config?''' [http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/lint.xml mobile/android/app/lint.xml]
 * '''open issues?''' Run locally for a complete list.