-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Inroduce service-provider
module
#29185
Conversation
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.
LGTM!
api(project(":base-services")) | ||
api(project(":core")) | ||
api(project(":core-api")) | ||
api(project(":service-provider")) |
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.
👏 That's what we want! Now let's get rid of that :core
dependency too :D
description = "An API for providing internal services for Gradle modules" | ||
|
||
dependencies { | ||
api(projects.javaLanguageExtensions) |
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.
❓ How come we don't need libs.inject
?
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.
It's not required by the classes currently moved into this new module
@@ -13,7 +13,6 @@ errorprone { | |||
} | |||
|
|||
dependencies { | |||
api(project(":base-services")) |
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.
👍 Yes!
9c4578f
to
206cc9e
Compare
@bot-gradle test without pts |
I've triggered the following builds with parameters: |
@bot-gradle test without pts |
I've triggered the following builds with parameters: |
plugins { | ||
id("gradlebuild.distribution.implementation-java") | ||
} | ||
|
||
description = "An API for providing internal services for Gradle modules" |
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.
ℹ️ Adding this new project is the main change
No code changes, only moves some classes into a new module.
Adds a new module that is home to the classes that allows other modules to register internal Gradle services.
This includes classes like
ServiceRegistration
andPluginServiceRegistry
.The
Scope
interfaces should later be moved here as well, because they participate in the service declaration.