Skip to content

Commit e028c59

Browse files
authored
Merge pull request github#14982 from michaelnebel/csharp/fixmissingpackageprinting
C#: Fix problem with logging unused packages.
2 parents 44be7df + aec070f commit e028c59

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,12 +354,13 @@ private IEnumerable<string> GetAllPackageDirectories()
354354

355355
return new DirectoryInfo(packageDirectory.DirInfo.FullName)
356356
.EnumerateDirectories("*", new EnumerationOptions { MatchCasing = MatchCasing.CaseInsensitive, RecurseSubdirectories = false })
357-
.Select(d => d.FullName);
357+
.Select(d => d.Name);
358358
}
359359

360360
private void LogAllUnusedPackages(DependencyContainer dependencies) =>
361361
GetAllPackageDirectories()
362362
.Where(package => !dependencies.Packages.Contains(package))
363+
.Order()
363364
.ForEach(package => progressMonitor.LogInfo($"Unused package: {package}"));
364365

365366
private void GenerateSourceFileFromImplicitUsings()

0 commit comments

Comments
 (0)