Skip to content

Commit 6b1ded3

Browse files
authored
Undo RemoveSharedFrameworkDependencies workaround
1 parent 8198eeb commit 6b1ded3

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

eng/tools/RepoTasks/RemoveSharedFrameworkDependencies.cs

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -64,17 +64,12 @@ private void FilterDependencies(string targetPath, ISet<string> dependencyToRemo
6464
var updatedGroup = new PackageDependencyGroup(group.TargetFramework, packages);
6565
foreach (var dependency in group.Packages)
6666
{
67-
// Hack to temporarily avoid using ISet.Contains
68-
if (!dependencyToRemove.Add(dependency.Id))
67+
if (dependencyToRemove.Contains(dependency.Id))
6968
{
7069
referencesFrameworkOnlyAssembly = true;
7170
Log.LogMessage($" Remove dependency on '{dependency.Id}'");
7271
continue;
7372
}
74-
else
75-
{
76-
dependencyToRemove.Remove(dependency.Id);
77-
}
7873

7974
packages.Add(dependency);
8075
}
@@ -112,4 +107,4 @@ private void FilterDependencies(string targetPath, ISet<string> dependencyToRemo
112107
}
113108
}
114109
}
115-
}
110+
}

0 commit comments

Comments
 (0)