nowinandroid

Commit Graph

Author	SHA1	Message	Date
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
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
Simon Marquis	1921c6544d	Replace deprecated methods	2 years ago
Alex Vanyo	1998fc1a20	Fix JVM target comment Change-Id: I9e62b796ed096f6cf586b51dc3a8e81f79c62e50	2 years ago
Takeshi Hagikura	e1bb03b33d	Set the source/target compatibility to 11 This is because up to Java 11 features are supported without requiring minimum API level nor AGP versions at the moment as in https://developer.android.com/studio/write/java11-minimal-support-table	2 years ago
Don Turner	e1ae12129a	Merge pull request #662 from android/av/agp-8.0 Update to AGP 8.0	2 years ago
Simon Marquis	80b8ae7a98	Don't rely on `finalizeDsl` API to configure Crashlytics	2 years ago
Alex Vanyo	4cdfaa93b2	Fix comment referring to 11 Change-Id: Ibef7bf42f60dd132535e70a7619958b242b7afe9	2 years ago
Alex Vanyo	e323c2155f	Remove unused imports in build-config Change-Id: I9fa6da0490fd9dc43cf5ed21aaa770c76b461a55	2 years ago
Alex Vanyo	38260e5a86	Update to VERSION_17 everywhere Change-Id: I3e75e8a833f45365657a24c4af652c2449af804c	2 years ago
Alex Vanyo	086dc957c0	Update to AGP 8.0 Change-Id: I27eb106e484e2003d732782a39612dc72c845282	2 years ago
Milosz Moczkowski	37908ec05c	Merge pull request #630 from drawers/david/delete-unnecessary-gradle-wrapper Remove unnecessary copy of Gradle wrapper	2 years ago
David Rawson	973eb04d81	Remove unnecessary Gradle wrapper	2 years ago
Simon Marquis	8ec54d19d8	Sort and group Gradle dependencies thanks to square/gradle-dependencies-sorter	2 years ago

1 2 3

132 Commits (1d0435832d785fa445931b5142cc4df6677a5e6a)