-
Notifications
You must be signed in to change notification settings - Fork 18
Gradle chores #4123
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
base: main
Are you sure you want to change the base?
Gradle chores #4123
Conversation
Automate adding the "-service" suffix to service projects to reduce the manual maintenance effort. Signed-off-by: Martin Nonnenmacher <[email protected]>
Automate adding the parent project name as a prefix to SPI projects to reduce the manual maintenace effort. Signed-off-by: Martin Nonnenmacher <[email protected]>
Before, only the `config-worker` project had the "-worker" suffix, add it to all workers for consistency. Signed-off-by: Martin Nonnenmacher <[email protected]>
Add the component name as a prefix to the names of all subprojects. This ensures that all subprojects and their generated JAR files have unique names. Fixes #4087. Signed-off-by: Martin Nonnenmacher <[email protected]>
|
Issues referenced in commit messages and issues linked to this PR are not in sync. |
| // Append "-service" to all service project names. | ||
| rootProject.children.single { it.name == "services" }.children.forEach { it.name = "${it.name}-service" } | ||
|
|
||
| // Append "-worker" to all worker project names. |
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.
Isn't this a breaking changes as it also changes the names of some published artifacts, I guess?
| commonMain { | ||
| dependencies { | ||
| api(projects.components.infrastructureServices.apiModel) | ||
| api(projects.components.infrastructureServices.infrastructureServicesApiModel) |
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.
Same here, isn't this a breaking change for consumers of the published artifacts?
See the commit messages for details.