Hey, all.

*tldr; `./mach gradle base:lint` or more generally, `./mach gradle <gradle
sub-project>:lint`**, and `./mach gradle lint`. Note that the standard
rules for running with Gradle should apply here (i.e. build, package, and
gradle-install first).*

Fun fact: Android Lint [1] "Just Works" with Gradle (shout out to
nalexander for working so hard to get Gradle up and running! \o/).

To run via the command line, you can use the commands above. I'm working on
supplying a proper configuration file [2], auditing our current lint
suppressions [3], and cleaning up the errors [4]. Note that because there
are so many errors, Intellij prints the results to HTML and XML documents -
see a sample (without CSS) at [5].

Under the hood, this has been running and marking errors in Intellij the
whole time, but with the efforts to clean up the warnings, we can start
enforcing the output and fixing bugs before they land - e.g. if lint fails,
the build fails! Because Intellij uses the same configuration file, we'll
miss warnings in Intellij if we start this practice now (though perhaps
there's some build magic to be done here...?).

If you'd like to get involved, either start work in a separate sub-project
(e.g. "app") and file some bugs, file a bug for the specific warning you'd
like to fix (e.g. "DrawAllocation") and CC me, or message me and we can
separate the work out.

I'll reply to this email as I make additional progress.

Happy (cleaner?) coding! ^_^
- Mike (:mcomella)

[1]: https://developer.android.com/tools/help/lint.html

[2]: https://bugzilla.mozilla.org/show_bug.cgi?id=1170285
[3]: https://bugzilla.mozilla.org/show_bug.cgi?id=1170886
[4]: https://bugzilla.mozilla.org/show_bug.cgi?id=1170884

[5]: https://bug1170285.bugzilla.mozilla.org/attachment.cgi?id=8614458
_______________________________________________
mobile-firefox-dev mailing list
[email protected]
https://mail.mozilla.org/listinfo/mobile-firefox-dev

Reply via email to