Confirmed users
975
edits
Mobile/Fennec/Android/Testing: Difference between revisions
→Static analysis
| Line 142: | Line 142: | ||
=== Static analysis === | === Static analysis === | ||
There are some other tools to be found at the [[Mobile/Fennec/Android/Testing/Not_yet_in_common_use|Not yet in common use page]]. | There are some other tools to be found at the [[Mobile/Fennec/Android/Testing/Not_yet_in_common_use|Not yet in common use page]]. | ||
==== checkstyle ==== | |||
* Run it in Intellij/Android Studio with [[Mobile/Fennec/Android/Static_analysis/checkstyle-idea|checkstyle-idea]]. | |||
* '''example checks?''' [http://checkstyle.sourceforge.net/google_style.html Google's style guide] | |||
* '''config?''' [http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/checkstyle.xml mobile/android/app/checkstyle.xml] | |||
* '''open issues?''' Issues we care about are blocking the meta | |||
* '''meta bug?''' [https://bugzilla.mozilla.org/show_bug.cgi?id=1170283 meta bug] | |||
==== Android Lint ==== | |||
* '''example checks?''' [https://sites.google.com/a/android.com/tools/tips/lint-checks `lint --show`] | |||
* '''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. | |||
* '''meta bug?''' [https://bugzilla.mozilla.org/show_bug.cgi?id=1170283 meta bug] | |||
* If you fix all issues for a warning, modify [http://mxr.mozilla.org/mozilla-central/source/mobile/android/app/lint.xml lint.xml] to change your warning into an error! | |||
==== eslint ==== | ==== eslint ==== | ||