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
Ensure all version catalog dependency properties are copied #23246
Ensure all version catalog dependency properties are copied #23246
Conversation
314a24b
to
729a325
Compare
6df8e55
to
ee87bfe
Compare
Reverted the I wrote up an issue to resolve the remaining two uncopied properties in 8.1: |
ee87bfe
to
833cfab
Compare
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.
Other than the comment about the test, LGTM
...t/groovy/org/gradle/integtests/resolve/catalog/VersionCatalogExtensionIntegrationTest.groovy
Outdated
Show resolved
Hide resolved
Follow up for #23096 to ensure all properties are properly copied
Even though these are desired changes, we are choosing to back them out of 7.6.1 as they are technically behavior changes and do not resolve an immediate 7.6 regression.
833cfab
to
10086f7
Compare
@bot-gradle test this |
I've triggered the following builds for you: |
@bot-gradle test and merge |
I've triggered a build for you. |
Ensures the 7.6 regression fixed in #23246 is included in 8.0 Co-authored-by: Justin Van Dort <jvandort@gradle.com>
Follow up for #23096 with a more comprehensive test.
Ensures
targetConfiguration
is copied when version catalog dependencies are copied. This property was copied pre7.6
but regressed in7.6
.While fixing this bug, I discovered
endorsing
andversionConstraint.branch
were also not copied. We are leaving those fixes out of this PR but will fix in8.1
. See: #23286