Skip to content

Commit 2f24162

Browse files
committed
C#: NoOpPackageRestore only needs the PackageConfigs collection and not the entire FileProvider.
1 parent f65d1e8 commit 2f24162

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

csharp/extractor/Semmle.Extraction.CSharp.DependencyFetching/PackagesConfigRestorer.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public static IPackagesConfigRestore Create(FileProvider fileProvider, Dependenc
4040
return new NugetExeWrapper(fileProvider, packageDirectory, logger, useDefaultFeed);
4141
}
4242

43-
return new NoOpPackagesConfig(fileProvider, logger);
43+
return new NoOpPackagesConfig(fileProvider.PackagesConfigs, logger);
4444
}
4545

4646
/// <summary>
@@ -343,15 +343,15 @@ public void Dispose()
343343
private class NoOpPackagesConfig : IPackagesConfigRestore
344344
{
345345
private readonly Semmle.Util.Logging.ILogger logger;
346-
private readonly FileProvider fileProvider;
346+
private readonly ICollection<string> packagesConfigs;
347347

348-
public NoOpPackagesConfig(FileProvider fileProvider, Semmle.Util.Logging.ILogger logger)
348+
public NoOpPackagesConfig(ICollection<string> packagesConfigs, Semmle.Util.Logging.ILogger logger)
349349
{
350-
this.fileProvider = fileProvider;
350+
this.packagesConfigs = packagesConfigs;
351351
this.logger = logger;
352352
}
353353

354-
public int PackageCount => fileProvider.PackagesConfigs.Count;
354+
public int PackageCount => packagesConfigs.Count;
355355

356356
public int InstallPackages()
357357
{

0 commit comments

Comments
 (0)