Skip to content

Commit 142a72c

Browse files
committed
C#: Address review comments.
1 parent c747352 commit 142a72c

1 file changed

Lines changed: 6 additions & 7 deletions

File tree

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

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,13 @@ internal interface IPackagesConfigRestore : IDisposable
2323
/// <summary>
2424
/// Factory for creating a package manager to restore NuGet packages referenced in packages.config files.
2525
/// If the environment doesn't support using nuget.exe to restore packages from packages.config files, a no-op implementation is returned.
26-
/// It is worth noting that for MacOS and Linux, nuget.exe is used with mono. However, mono is being deprecated and the last images to contain
27-
/// mono are
26+
/// It is worth noting that for macOS and Linux, nuget.exe is used with mono. However, mono is being deprecated and the last GitHub images
27+
/// to contain mono are:
2828
/// - Ubuntu 22.04
29-
/// - MacOS 14
30-
///
31-
/// It is worth noting that even with the removal of mono, the content of the packages.config files are parsed and added to the packages list in
32-
/// the FileContent implementation. If the packages are not restored in this step, there is a subsequent step that still may succeed in
33-
/// restoring the packages, albeit without the help of nuget.exe.
29+
/// - macOS 14
30+
///
31+
/// If the packages from the packages.config files are not restored with the packages.config restore functionality below, there is a subsequent
32+
/// step that still may succeed in restoring the packages without the help of nuget.exe (by attempting to restore using dotnet).
3433
/// </summary>
3534
internal class PackagesConfigRestoreFactory
3635
{

0 commit comments

Comments
 (0)