nowinandroid

Commit Graph

Author	SHA1	Message	Date
Simon Marquis	eae51a6df9	[Fix] Add missing unit test dependency for JVM modules `:core:common` module would not be able to run its unit test: `:core:common:test` Related to: #1546	1 year ago
Simon Marquis	8ea9a8fa51	Android Lint improvements - Create `LintConventionPlugin` to configure Lint on all compatible modules: Android applications, Android libraries and JVM modules. - Run `lintProdRelease` in CI instead of the default `lintDemoDebug` which is less important compared to production code. - Rearrange CI steps to make it more clear that Lint should is an additional step after build (build -> test -> lint). - Enable SARIF support and upload results to GitHub's CodeQL to get inline feedback on PRs. If we really need better perfs, we could restore `lint.checkDependencies = true` on the `:app` module and only execute `:app:lintProdRelease`. But in practice, this does not change the total build time on this project.	3 years ago
Alex Vanyo	76b0b08121	Adjust JDK setup to remove toolchain Change-Id: Ie0f43cb8f0fe504ed3e98ef35703949959688699	3 years ago

Author

SHA1

Message

Date

Simon Marquis

eae51a6df9

[Fix] Add missing unit test dependency for JVM modules

`:core:common` module would not be able to run its unit test: `:core:common:test`

Related to: #1546

Simon Marquis

8ea9a8fa51

Android Lint improvements

- Create `LintConventionPlugin` to configure Lint on all compatible modules: Android applications, Android libraries and JVM modules.
- Run `lintProdRelease` in CI instead of the default `lintDemoDebug` which is less important compared to production code.
- Rearrange CI steps to make it more clear that Lint should is an additional step after build (build -> test -> lint).
- Enable SARIF support and upload results to GitHub's CodeQL to get inline feedback on PRs.

If we really need better perfs, we could restore `lint.checkDependencies = true` on the `:app` module and only execute `:app:lintProdRelease`.
But in practice, this does not change the total build time on this project.

Alex Vanyo

76b0b08121

Adjust JDK setup to remove toolchain

Change-Id: Ie0f43cb8f0fe504ed3e98ef35703949959688699

3 Commits (eae51a6df932ec0f0634e10d810fa213b789b36b)