127 Commits (fe5bfdeccfccae82766fef58609d943bbda0738d)

Author	SHA1	Message	Date
Simon Marquis	17162dab60	graphUpdate	3 months ago
Simon Marquis	1c61be7b7b	Update color mapping ... | Type | Plugin ID | Color | |-------------|------------------------------------|------------------------| | Application | `nowinandroid.android.application` | `#CAFFBF` PastelGreen | | Feature | `nowinandroid.android.feature` | `#FFD6A5` PastelOrange | | Library | `nowinandroid.android.library` | `#9BF6FF` PastelBlue | | Test | `nowinandroid.android.test` | `#A0C4FF` PastelMauve | | JVM | `nowinandroid.jvm.library` | `#BDB2FF` PastelPurple | | Unknown | `?` | `#FFADAD` PastelRed |	3 months ago
Simon Marquis	e8dc43b58b	graphUpdate	4 months ago
Simon Marquis	e079e68815	Delete old graphs	4 months ago
Aurimas Liutikas	7ff1913855	Fixes host test tasks that were accidentally attemping to run ... Gradle 9.0.0 has the new enforcement that Test task runs that have sources but no tests run should fail. Without this change we get Execution failed for task ':core:analytics:testDemoDebugUnitTest'. > There are test sources present and no filters are applied, but the test task did not discover any tests to execute. This is likely due to a misconfiguration. Please check your test configuration. If this is not a misconfiguration, this error can be disabled by setting the 'failOnNoDiscoveredTests' property to false. We get this because before this change AndroidCompose was always setting isIncludeAndroidResources = true which generates a source to the test task and in the case of :core:analytics it is the only source. Instead of setting isIncludeAndroidResources in AndroidCompose move to setting it in relevant build.gradle.kts files (most already had it)	5 months ago
hoangchungk53qx1	3c5026065d	Refactor: Replace list initialization with emptyList() for clarity	6 months ago
ogapants	ef4b2eaddd	Replace org.junit.Assert.assertEquals with kotlin.test.assertEquals for consistency	9 months ago
Don Turner	ef0d89e9fb	Update core/data/src/test/kotlin/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstNewsRepositoryTest.kt	12 months ago
E Y H	fa532089e2	Update OfflineFirstNewsRepositoryTest.kt ... Remove the comments. A similar comment wasn't accpeted in my last PR.	12 months ago
EYH GPT	798b941b75	Update OfflineFirstNewsRepositoryTest.kt ... Improve the unit test to perform meaningful comparisons instead of merely comparing two empty lists.	12 months ago
Don Turner	b7973eeac9	Fix spotless	12 months ago
Don Turner	793317bfbd	Update core/data/src/test/kotlin/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstTopicsRepositoryTest.kt	12 months ago
Don Turner	dfc55c94e3	Update core/data/src/test/kotlin/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstTopicsRepositoryTest.kt	12 months ago
Pirate Cat	b651b2ed4d	Update OfflineFirstTopicsRepositoryTest.kt ... Improve the unit test to do meaning comparisons instead of just comparing two empty lists.	12 months ago
Don Turner	fd3aeb56dc	Merge pull request #1442 from Jaehwa-Noh/remove-default-value ... Remove `isReturnDefaultValues` to debug easy.	1 year ago
Don Turner	18bb7a45a2	Fix spotless	1 year ago
Don Turner	717e0c7ad0	Add unit test for NetowrkTopic mapper	1 year ago
Don Turner	b478ceae54	Fix failing UI test, add unit test and refactor	1 year ago
Simon Marquis	dac9a6a0fa	Remove implicit `TemporaryFolder` binding with InMemoryDataStore test module	1 year ago
Simon Marquis	d56e06e676	Merge Android & JVM plugins into a single `HiltConventionPlugin`	1 year ago
Jaehwa Noh	8560632efd	Remove `isReturnDefaultValues`. ... Change-Id: I028244a23b6215e21269602b8df07e58b136b9f1	2 years ago
Tomáš Mlynarič	c396352d83	Remove forgotten Trace.endSection() ... Change-Id: Ib6f7678f5f1c4b0f93f1736b6453c649d6f7dc97	2 years ago
Tomáš Mlynarič	6b35cbb328	Offload connectivity monitor to a background thread ... Change-Id: I9a2ef7766ae6abc6d8a7c86a4b49ef3c795e446c	2 years ago
Don Turner	ecdfdef4e4	Merge pull request #1291 from Jaehwa-Noh/rename ... Rename Fake to Demo	2 years ago
Simon Marquis	ee8797f940	Delete unused `combine` extension in `SyncUtilities.kt`	2 years ago
Jaehwa Noh	27ca860745	Rename fake to demo. ... Change-Id: I5b1d7963e623832a766900aa74ef1085aac0cacc	2 years ago
Alex Vanyo	78c77737c0	Merge pull request #1275 from Jaehwa-Noh/rename-function ... Rename function to setNewsResourceBookmarked	2 years ago
Don Turner	a5025390af	Add title to READMEs to describe the dependency graph ... Change-Id: Ifb4b44e67df53ddd8ed048122c6c12a904e61528	2 years ago
Jaehwa Noh	ee4320723f	Fix typo. ... Change-Id: Id3aed2e3c54660fbbb42bc0c08dea68de0d5230f	2 years ago
Jaehwa Noh	ed91723e2f	Fix #1270 ... Change-Id: Id9774ad300b04c17cb1d7668c1ab1bac014bf70c	2 years ago
Don Turner	80a82de945	Update module dependency graphs and add script to create them ... Change-Id: I7b2ae264ec84491c847a44869934e2f238564c31	2 years ago
Don Turner	350e7b1bba	Merge pull request #1144 from SimonMarquis/move-fakes-to-data-test-module ... Move `:core:data` fake repositories to `:core:data-test`	2 years ago
Jaehwa Noh	52c0d7ae0a	Remove Insert function ... - Insert is only used in test. - Upsert do same thing as Insert. Change-Id: Ib5f8e693e71c895b83e43f6b51cebdc698bf744b	2 years ago
Tomáš Mlynarič	9daedc9690	Improve converting to kotlin timezone ... Change-Id: I2af929c341cb274ae6b93bb8dffe696de6ed1c63	2 years ago
Tomáš Mlynarič	12a3a6edde	Use trySend multiple times ... Change-Id: Ie00c4180d07fc4bb9d457647f3b7ddadd5a3c032	2 years ago
Simon Marquis	3d339e1658	Move `:core:data` fake repositories to `:core:data-test` ... Now that these fake implementations are no longer used in the production source code (only in tests), we can safely extract them out of this module. Hopefully, we'll later be able to use Kotlin testFixtures for that. References: - #709 - #875	2 years ago
Tomáš Mlynarič	6ec3b3c7a2	Use androidx.tracing everywhere ... Change-Id: I9a77c4434d271e2e23700823f9f513aa61d177f8	2 years ago
Tomáš Mlynarič	ea1573fcc1	Use kotlinx.datetime.Timezone instead of java ZoneId ... Change-Id: I62c4d044b319a9b59e06bb42fd12971d992e8628	2 years ago
Tomáš Mlynarič	a16d5f726c	More readable zoneId creation ... Change-Id: I912dc021554e4886a5d09e63799aa5c81e8fd16a	2 years ago
Tomáš Mlynarič	1e925ed99f	Conflate TimeZoneMonitor emissions ... Co-authored-by: Alex Vanyo <vanyo@google.com>	2 years ago
Tomáš Mlynarič	cb00d2c8cb	Add TimeZoneMonitor to prevent multiple TimeZoneBroadcastReceivers ... This way, we can save ~1ms per composed item on screen. Change-Id: Ib9ada3cea53304fca4fb2b36c48c175845bc683d	2 years ago
Don Turner	707117a4f2	Merge branch 'main' into kotlinify	2 years ago
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`	2 years ago
Simon Marquis	caa482bc71	Kotlinify codebase ... - Remove unnecessary nullable types - Replace no-op method bodies with Unit - Convert to expression body - Replace if with when - Remove braces from 'when' entries - Remove braces from if statement - Convert to single line lambda - oneline if/returns - Replace 'contains' call with 'in' operator Following this refactor, it could be great to envision a more "strict" code formatter like ktlint 1.0 (we are currently stuck at 0.48.1)	2 years ago
Don Turner	a381d13199	Merge pull request #958 from McGalanes/main ... Use typesafe project accessors	2 years ago
Melwin Magalhaes	6247712064	Add convention plugins to version catalogs	2 years ago
Melwin Magalhaes	b1266dc05a	Use typesafe project accessors for project modules	2 years ago
Simon Marquis	a33dafccad	Migrate from `java` to `kotlin` sourceSet directory	2 years ago
Don Turner	d0909a9c8f	Merge pull request #925 from SimonMarquis/remove-dispatcher ... Remove unnecessary dispatcher switch in `DefaultRecentSearchRepository`	2 years ago
Simon Marquis	0ddcf0b4ab	Remove unnecessary dispatcher switch in `DefaultRecentSearchRepository` ... As mentioned in the following discussion: https://github.com/android/nowinandroid/discussions/818 Room already takes care of switching dispatchers if the DAO method is correctly annotated with the `suspend` modifier.	2 years ago