-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Merge changes 'refactor modules dependencies' from TCS #117
Conversation
9e7917f
to
e848c89
Compare
e848c89
to
b2fa4e9
Compare
f8d1f83
to
f4d823f
Compare
build-settings/conventions/src/main/kotlin/convention-plugins.settings.gradle.kts
Outdated
Show resolved
Hide resolved
|
||
} | ||
repositories { | ||
google() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need google and gradle portal repositories for most of the plugins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
somehow, due to magic of settings plugins, we need these ones in application settings.gradle.kts
application/build.gradle.kts
Outdated
) | ||
) | ||
} | ||
|
||
|
||
tasks.register("build") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets add some comments since its not obvious why we need this tasks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this tasks?
it is a lifecycle task. it's just to be able to call root project task in this way: ./gradlew :build
which is the synonym ./gradlew build
- Remove root project tasks, cause they are not needed - Remove empty plugins block in convention-plugins.settings.gradle.kts - Remove doubling rootProject.name - Move includeBuild build-dependencies out of pluginManagement block - Add includeBuild plugins cause we apply tools.forma.android and tools.forma.deps as settings plugin
Source here tinkoff-mobile-tech#1