Skip to content
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

change groovy dependency to fixed version so we don't get the beta r… #43

Merged
merged 1 commit into from
Sep 6, 2017

Conversation

buildlackey
Copy link
Contributor

Here are the full details of why we needed to update the dependency on groovy all:

The Groovy project released pre-release dependencies of 2.5.0 to Maven Central/JCenter in the last few days. Teams with a 2.+ or latest.release constraint on these dependencies are introducing them into the dependency graph, causing downstream projects to break.

A resolution rule has been added to reject these versions. For projects with first order dependencies, the rule will ensure that a 2.4.x version is selected. Downstream projects will not be able to resolve these versions, and will fail with:

Could not find org.codehaus.groovy:groovy-all:2.5.0-beta-1.
Required by:
project :recs-test > netflix:p13nalgo-recs-client-esl:2.116 > netflix:videometadata-client:60.29

@buildlackey buildlackey requested review from vinaykumarchella and ipapapa and removed request for vinaykumarchella September 6, 2017 20:42
@ipapapa ipapapa merged commit 9a65415 into Netflix:master Sep 6, 2017
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