-
Notifications
You must be signed in to change notification settings - Fork 200
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
Feature/current version in component selection #327
Feature/current version in component selection #327
Conversation
That’s a very elegant solution, thanks! I’m not very strong at Groovy either. They do have syntax for automatic delegations which might be more idiomatic. I’d keep to 100c line limit, add the latest Gradle versions for integration testing, and add your enhancement to the readme. Otherwise it lgtm. |
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.
I sent you the commit access, feel free to merge when ready :)
I did a little testing of this and found a slight bug. If you bump the version and run Gradle has internal, hidden dependencies like jacoco and ant. These don't have a version number, nor do cases like local jars. It appears that this causes an NPE in your changes. Can you take a quick look?
|
I just had a look and you are right. There is no current version for eg. jacoco. For these I also see "Skipping hidden dependency", so they do not appear in the output. So I guess the best solution would be filtering those, so they do not get into componentSelection block at all. I will prepare a quick fix. Thanks for the tip on |
This is released! 😺 |
After facing similar issue as in #313 and #290, I had a look at how we could get current version to resolution strategy.
I went for extending default gradle befavior by decorating the resolution strategy classes. Please let me know if the approach would be acceptable. As for code style, I am not much at home in groovy, so any improvements are welcome.