Adjust comments based on PR feedback

dotnet · Jan 25, 2018 · 4eae2d7 · 4eae2d7
1 parent b81d54d
commit 4eae2d7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/Tasks/Microsoft.NET.Build.Tasks/LockFileExtensions.cs b/src/Tasks/Microsoft.NET.Build.Tasks/LockFileExtensions.cs
@@ -100,6 +100,7 @@ public static Dictionary<string, string> GetProjectFileDependencies(this LockFil
 
         public static HashSet<string> GetProjectFileDependencySet(this LockFile lockFile)
         {
+            // Get package name from e.g. Microsoft.VSSDK.BuildTools >= 15.0.25604-Preview4
             string GetPackageNameFromDependency(string dependency)
             {
                 int indexOfWhiteSpace = IndexOfWhiteSpace(dependency);

diff --git a/src/Tasks/Microsoft.NET.Build.Tasks/build/Microsoft.PackageDependencyResolution.targets b/src/Tasks/Microsoft.NET.Build.Tasks/build/Microsoft.PackageDependencyResolution.targets
@@ -379,7 +379,7 @@ Copyright (c) .NET Foundation. All rights reserved.
       <!--
         Update Reference items with NuGetPackageId metadata to set ExternallyResolved appropriately.
         NetStandard.Library adds its assets in targets this way and not in the standard way that
-        would get ExternallyResolved set in ResolvedCompileFileDefinitions below.
+        would get ExternallyResolved set in ResolvePackageAssets.
        -->
       <Reference Condition="'%(Reference.NuGetPackageId)' != ''">
         <ExternallyResolved>true</ExternallyResolved>