Skip to content

Improve usage of Provider API#288

Merged
rpalcolea merged 4 commits intomainfrom
plugin-enhancements
Dec 1, 2025
Merged

Improve usage of Provider API#288
rpalcolea merged 4 commits intomainfrom
plugin-enhancements

Conversation

@rpalcolea
Copy link
Copy Markdown
Member

@rpalcolea rpalcolea commented Dec 1, 2025

GitBuildService now uses Providers everywhere instead of eagerly executing git commands during configuration. Converted the remote field from String to Property and swapped out the old Groovy Closure DSL for Action<>.

Also made the ReleasePlugin constructor lazy by not calling .get() on the gitBuildService provider until we actually need it.

* Configures the tag strategy with the provided action.
*/
void tagStrategy(Closure closure) {
ConfigureUtil.configure(closure, tagStrategy)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

project.plugins.withType(IvyPublishPlugin) {
project.publishing {
publications.withType(IvyPublication) {
publications.withType(IvyPublication).configureEach {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@wakingrufus wakingrufus self-requested a review December 1, 2025 23:47
@rpalcolea rpalcolea merged commit d017424 into main Dec 1, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants