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
GH787: Enhance NuGet dependency handling. #1848
Conversation
bjorkstromm
commented
Sep 28, 2017
- Fixes Reference NuGet dependencies installed via the #addin directive #787
- Don't share project. Only share Gather Cache.
- Use DependencyBehavior.Ignore by default. Only set Lowest if load dependencies is set. This way we wont install unneccessary packages and cache gathering is faster.
@Redth are you also interested in reviewing? This addresses the shared |
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.
A minor suggestion, but nothing that would make me request changes. 👍
{ | ||
if (type == PackageType.Addin) | ||
{ | ||
return package.ShouldLoadDependencies(_config) ? |
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.
You could change this to
if(package.ShouldLoadDependencies(_config))
{
return DependencyBehavior.Lowest;
}
since the default behavior is to ignore dependencies by exiting the scope. Just a personal preference of course. Do as you wish 😄
LGTM! |
- Fixes cake-build#787 - Don't share project. Only share Gather Cache. - Use DependencyBehavior.Ignore by default. Only set Lowest if load dependencies is set. This way we wont install unneccessary packages and cache gathering is faster.
c7471fc
to
c6cfe2f
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.
LGTM 👍 I've rebased and will merge as soon as CI finishes.
@mholo65 your changes have been merged, thanks for your contribution 👍 |
And how to use that feature? I still have ICSharpCode.SharpZipLib, Version=0.86.0.518 not resolved. |
The nuget optin configuration is documented here: |