Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Goodbye Dagger, Hello DependencyProperty #1

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from

Conversation

wada811
Copy link
Owner

@wada811 wada811 commented Jul 9, 2020

As a result of a simple build speed measurement, the build time was reduced by about 10 seconds when migrating from Dagger.

Dagger DependencyProperty Compare
Task execution 1m 22.757s 1m 8.773s 14s(17%) faster
- KaptGenerateStubsTask 32.145s 16.439s 16s(50%) faster
- KaptWithKotlincTask 55.888s 37.388s 18s(33%) faster
- KotlinCompileWithWorkers 55.213s 43.037s 12s(21%) faster

The measurement was performed several times, and representative results were extracted and compared.

./gradlew clean cleanBuildCache
./gradlew --scan assembleDebug

@wada811 wada811 force-pushed the feature/goodbye-Dagger branch 4 times, most recently from b5c607f to cc11984 Compare July 11, 2020 14:29
@wada811 wada811 force-pushed the feature/goodbye-Dagger branch 4 times, most recently from e6f7b38 to 1a9b332 Compare July 11, 2020 14:37
@wada811 wada811 closed this Jul 11, 2020
@wada811 wada811 reopened this Jul 11, 2020
@wada811 wada811 force-pushed the feature/goodbye-Dagger branch 2 times, most recently from 88f3fe1 to d5cea5c Compare July 13, 2020 13:10
@wada811 wada811 changed the title Goodbye Dagger Goodbye Dagger Hilt, Hello DependencyProperty Sep 10, 2020
@wada811 wada811 changed the title Goodbye Dagger Hilt, Hello DependencyProperty Goodbye Dagger, Hello DependencyProperty Sep 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant