-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
allow to install multiple versions of the same packages #891
Conversation
public static class IPackageInstallationServiceExtensions | ||
{ | ||
[Time] | ||
public static async Task<InstallerResult> InstallAsync( |
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.
This is a breaking change and requires recompilation of the app.
@@ -0,0 +1,17 @@ | |||
namespace Orc.NuGetExplorer.Models; |
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.
Remove .Models
{ | ||
// Track packages which already installed and make sure only one version of package exists | ||
var resolver = new Resolver.PackageResolver(); | ||
availablePackagesToInstall = await resolver.ResolveWithVersionOverrideAsync(resolverContext, project, DependencyBehavior.Ignore, |
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.
Dependency behavior is changed here, which is not good
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.
oops, sorry
forgot to change it back
Thanks for review guys |
No description provided.