Skip to content

Update Gradle dependencies to use implementation#10688

Merged
artembilan merged 1 commit intospring-projects:mainfrom
cppwfs:SI-ref-comp-imp-upd
Jan 9, 2026
Merged

Update Gradle dependencies to use implementation#10688
artembilan merged 1 commit intospring-projects:mainfrom
cppwfs:SI-ref-comp-imp-upd

Conversation

@cppwfs
Copy link
Contributor

@cppwfs cppwfs commented Jan 9, 2026

Replace deprecated compile configuration with implementation across all Spring Integration reference documentation. The compile configuration was deprecated in Gradle 4.0 and removed in Gradle 7.0. This update ensures documentation examples use current Gradle best practices.

Replace deprecated `compile` configuration with `implementation`
across all Spring Integration module documentation. The `compile`
configuration was deprecated in Gradle 4.0 and removed in Gradle 7.0.
This update ensures documentation examples use current Gradle best
practices.
@cppwfs cppwfs requested a review from artembilan January 9, 2026 12:07
@cppwfs cppwfs added this to the 7.1.0-M1 milestone Jan 9, 2026
@artembilan
Copy link
Member

Thanks.
I’m not sure how is that useful , but we should use relevant labels on issues or PRs.

@cppwfs
Copy link
Contributor Author

cppwfs commented Jan 9, 2026

compile is deprecated on older versions of gradle does not work on the latest gradle version. So why would we keep the syntax compile when the recommendation is to use implementation?

@artembilan artembilan merged commit 44db51b into spring-projects:main Jan 9, 2026
2 of 3 checks passed
@artembilan
Copy link
Member

I’m not sure in your question.
I fully agreed with the change, so it has been merged.
What I meant is to add a documentation label to the PR since it is indeed about docs.
Just some issues management.

@cppwfs
Copy link
Contributor Author

cppwfs commented Jan 9, 2026

I see now. Thank you for the merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants