144 Commits (2ba04dfd9d88efe35f72865aeb1dda33707aa7e9)

Author	SHA1	Message	Date
Simon Marquis	f27a2e176e	Reduce overall build complexity ... - Reduce the visibility of multiple Hilt `Module`s and implementations of public interfaces - Correctly configure the visibility of dependencies: - `api` when it's part of it's public `api` - `implementation` when it's part of it's internal `implementation` - Remove unnecessary dependencies in build.gradle.kts files - Remove unnecessary dependencies provided by plugins - Remove unnecessary applied plugins (i.e. roborazzi) - Sort dependencies in `build.gradle.kts` - Delete unused entries in `libs.versions.toml`	12 months ago
Alex Vanyo	46b0a81080	Use normal propery setting ... Change-Id: I31aef2007b63c067ad80a9f65f2b9594d114f13a	12 months ago
Alex Vanyo	8a992996a4	Enable stricter validation for build-logic convention plugins ... Change-Id: I1955393c76e91bebfe4b7e349734c81a10263c8c	12 months ago
Jonathan Koren	392eaa3e61	Add dependencyGuard to AndroidApplicationConventionPlugin ... Change-Id: Ib695ca18512d0ccde4c67e101f09ebbff8897912	1 year ago
Daniel Santiago	c908f989c8	Update to Hilt 2.48.1 and use the KSP version ... * Moved protobuf to separate module to avoid having to wire KSP's Plugin with the Protobuf Plugin. * `@Binds` function cannot be an extension function. * Commented `kotlin.compiler.execution.strategy=in-process` in CI to circumvent OOMs due to increased metaspace usage.	1 year ago
Don Turner	f414cd482e	Merge pull request #1013 from android/av/sdkconstants-for-aapt2 ... Use SdkConstants for getting aapt2 executable location	1 year ago
Alex Vanyo	c70049edd8	Use specific task name for updating badging ... Change-Id: I52b75fb3ed2ae3a76656c211b7aee1aa521f1112	1 year ago
Alex Vanyo	c730cf20fa	Use SdkConstants for getting aapt2 executable location ... Change-Id: Ibc9ece0c3ade51a29954caee3af3a669ed99c384	1 year ago
lihenggui	66cb30a811	Fix #1008: Use capitalized variant name when registering gradle tasks	1 year ago
Alex Vanyo	360b33eb82	Add automatic checks for badging ... Change-Id: Ic8fca86d7aa84675fa91fb1aa99abcc6e19d663e	1 year ago
Don Turner	6e96d30519	Merge pull request #912 from lihenggui/main ... Upgrade Gradle to v8.3	1 year ago
Jose Alcérreca	151f877bbe	Adds screenshot tests to :core:designsystem (#931) ... * WIP: Adds screenshot tests to :core:designsystem Change-Id: I0672845feba4064652dd8d60f07047b87864e121 * 🤖 Updates screenshots * Creates tests for more components and cleans up Change-Id: I61fe3ae6a4e8a41a599d520e16fc14aa6a643a22 * WIP: More cleanup and more combinations of themes Change-Id: I34312bc7d147b31f1c638cd505a9c241f8267523 * Added the rest of the screenshot tests for designsystem Change-Id: Ic427db5491910781c038882055524e3f3dbed194 * Some more cleanup Change-Id: I7384e55864719af9122ad9da8e50a09cb9a60180 * Spotless Change-Id: I22aa46e1f56b8b638c9d609ababbe49d471a26c6 * 🤖 Updates screenshots * ScreenshotHelper cleanup Change-Id: Ic94d41618e7850ab47f294d8022b405f18c843f0	1 year ago
Simon Marquis	eadcbe6795	Fix typo in `AndroidCompose.kt`	1 year ago
Milosz Moczkowski	c57fb78fa7	Merge pull request #759 from SimonMarquis/lint/sarif ... Android Lint improvements	1 year ago
Jose Alcérreca	886158d3cb	Adds Screenshot testing with Roborazzi (#876) ... * Adds screenshot tests using Roborazzi (Robolectric Native Graphics) - Adds Roborazzi to convention plugins - Adds Screenshot helper in :core-testing - Creates screenshot suites for :app and :feature-foryou * CI and spotless * Moves :app tests to testDemo and makes NiaAppScreenSizesScreenshotTests prettier * CI: Moves local tests to their own step * CI: Adds --rerun to screenshot task * CI: Moves screenshots before local tests * CI: Fixes wrong if statement in workflow * CI WIP: trying to trigger the push step * CI: Re-enables roborazzi verification * Fixes flaky screenshot tests by setting LocalInspectionMode on * CI: screenshot commits now use the original author intead of bot account * CI: Disables globbing because file_pattern didn't work * CI: Trying new file pattern for png files * CI: Adds a check for forks * 🤖 Updates screenshots * Code review: toml cleanup, comments * Use new github.event.pull_request.head.repo.fork Co-authored-by: Simon Marquis <contact@simon-marquis.fr> * Uses Robolectric qualifiers to set the dpi, adds section to README * Spotless * Delegates creation of repository to Hilt in test * Revert "Use new github.event.pull_request.head.repo.fork" * 🤖 Updates screenshots * Empty commit to trigger GHA on main branch * Makes time zones deterministic in screenshot tests * Increases GMD timeout to 90m, but it has to be reduced --------- Co-authored-by: Simon Marquis <contact@simon-marquis.fr>	1 year ago
lihenggui	fcc49d141d	Upgrade Gradle to v8.3	1 year ago
Don Turner	492697ce6b	Merge pull request #873 from SimonMarquis/misc-warnings ... Fix miscellaneous warnings	1 year ago
Don Turner	246200f163	Merge pull request #754 from android/tm/metrics-one-folder ... Compose compiler metrics saved into one folder	1 year ago
Don Turner	e96e54f7de	Merge pull request #806 from SimonMarquis/objenesis ... Remove forced resolution strategy	1 year ago
Simon Marquis	4bdc9705af	Fix miscellaneous warnings ... - Remove unused namespace declaration - Collapse empty manifest tags - Remove redundant qualifier name - Remove redundant suppression	1 year ago
Don Turner	3756e3ea2d	Merge pull request #832 from SimonMarquis/patch-2 ... Align `targetSdk` to 34 in `AndroidTestConventionPlugin.kt`	1 year ago
Simon Marquis	61f7eb8823	Update new `CommonExtension` signature	1 year ago
Simon Marquis	8b7f6303c8	Align `targetSdk` to 34 in `AndroidTestConventionPlugin.kt` ... Initial update was applied in https://github.com/android/nowinandroid/pull/814	1 year ago
Murat Yener	f04652e56c	Merge branch 'baseline-profile-api-change' of https://github.com/android/nowinandroid into baseline-profile-api-change	1 year ago
Simon Marquis	d136965a0d	Merge branch 'main' into lint/sarif	1 year ago
Simon Marquis	cc9b5a1634	Remove SARIF support	1 year ago
Murat Yener	3a4ff15535	Merge branch 'baseline-profile-api-change' of https://github.com/android/nowinandroid into baseline-profile-api-change	1 year ago
Simon Marquis	0a3069a0d8	Remove forced junit resolution strategy ... By looking at the Gradle scan's dependency tab, junit 4.12 is already replaced by 4.13.2. There is no need to manually force this at the configuration level.	1 year ago
Simon Marquis	d332f7b70d	Remove forced objenesis resolution strategy ... The linked issue is marked as resolved, and not able to reproduce the mentioned compilation error.	1 year ago
Simon Marquis	a920baf224	Extract `Project.libs` extension for `build-logic` plugins	1 year ago
Simon Marquis	8e708cca05	Merge branch 'main' into lint/sarif	1 year ago
Simon Marquis	cdbac9379f	Revert "Remove `ExperimentalCoroutinesApi` and `FlowPreview` opt-in"	2 years ago
Milosz Moczkowski	1345764649	Merge pull request #770 from android/refactor/remove_unused_statements ... Remove unused statements	2 years ago
Miłosz Moczkowski	5bf66739bd	Remove redundant statements ... Change-Id: Iaa25b7ba033d5e9c85cc2dc433197ed7b8b6cb89	2 years ago
Tomáš Mlynarič	375480a2f9	Make paths relative ... Change-Id: Iefdf9b2a0313bbdc0532450bb10ca9b7cca41014	2 years ago
Simon Marquis	04ddc86298	Enable `checkDependencies` and run lint tasks on project roots	2 years 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.	2 years ago
Simon Marquis	502f9ed110	Remove `ExperimentalCoroutinesApi` and `FlowPreview` opt-in ... ``` > Task :core:model:compileKotlin w: Opt-in requirement marker kotlinx.coroutines.ExperimentalCoroutinesApi is unresolved. Please make sure it's present in the module dependencies w: Opt-in requirement marker kotlinx.coroutines.FlowPreview is unresolved. Please make sure it's present in the module dependencies ```	2 years ago
Tomáš Mlynarič	f577fb06cc	Add project path subfolder ... Change-Id: I1b680a37ecbd80b894b0d1b649633936cbe41328	2 years ago
Tomáš Mlynarič	d549ca6c7a	Apply suggestions from code review ... Co-authored-by: Simon Marquis <contact@simon-marquis.fr>	2 years ago
Tomáš Mlynarič	9f6a94dee2	Change folder for compose-reports ... Change-Id: I7757f13c36c60991586224707f91f8ac8c0beae8	2 years ago
Neelansh Sahai	02bf1ac230	Merge pull request #725 from tsumuchan/main ... Clarify dependencies of the included build-logic	2 years ago
Alex Vanyo	76b0b08121	Adjust JDK setup to remove toolchain ... Change-Id: Ie0f43cb8f0fe504ed3e98ef35703949959688699	2 years ago
tsumuchan	e1025d8363	Update gradle plugin naming	2 years ago
Milosz Moczkowski	88b9e023d0	Merge pull request #692 from android/av/fix-jvm-target-comment ... Fix JVM target comment	2 years ago
Simon Marquis	ddabba8e4a	Merge branch 'main' into jvmToolchain	2 years ago
Simon Marquis	4550743fac	Create and use CI specific group of GMD devices ... This should reduce the total CI time of `AndroidCIWithGmd` workflow. Using the matrix strategy at the GitHub Actions level forces us to run the tests sequentially and download/rebuild everything from scratch at each iteration.	2 years ago
Don Turner	2fd1b716f6	Merge pull request #667 from SimonMarquis/cleanup/warnings ... Cleanup warnings and replace deprecated methods	2 years ago
Simon Marquis	57d1bed2cc	Disable unnecessary Android instrumented tests	2 years ago
Simon Marquis	f19dd06200	Remove deprecated `kotlin.Experimental` opt-in ... This would produce the following warning messages: ``` w: Opt-in requirement marker kotlin.Experimental is unresolved. Please make sure it's present in the module dependencies ```	2 years ago