Namespace refactor for the nodejs integration extensions

CommunityToolkit · Sep 17, 2024 · 9367190 · 9367190
1 parent 9d547b0
commit 9367190
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 8 deletions.
diff --git a/examples/nodejs-ext/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions.AppHost/Program.cs b/examples/nodejs-ext/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions.AppHost/Program.cs
@@ -1,5 +1,3 @@
-using CommunityToolkit.Aspire.Hosting.NodeJS.Extensions;
-
 var builder = DistributedApplication.CreateBuilder(args);
 
 builder.AddViteApp("vite-demo")

diff --git a/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodeJSHostingExtensions.cs b/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodeJSHostingExtensions.cs
@@ -1,9 +1,10 @@
-using Aspire.Hosting;
-using Aspire.Hosting.ApplicationModel;
+using Aspire.Hosting.ApplicationModel;
 using Aspire.Hosting.Lifecycle;
+using Aspire.Hosting.Utils;
+using CommunityToolkit.Aspire.Hosting.NodeJS.Extensions;
 using Microsoft.Extensions.Hosting;
 
-namespace CommunityToolkit.Aspire.Hosting.NodeJS.Extensions;
+namespace Aspire.Hosting;
 
 public static class NodeJSHostingExtensions
 {

diff --git a/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodePackageInstaller.cs b/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/NodePackageInstaller.cs
@@ -5,14 +5,13 @@
 
 namespace CommunityToolkit.Aspire.Hosting.NodeJS.Extensions;
 
-
 /// <summary>
 /// Represents a Node package installer.
 /// </summary>
 /// <param name="packageManager">The package manager to use.</param>
 /// <param name="loggerService">The logger service to use.</param>
 /// <param name="notificationService">The notification service to use.</param>
-class NodePackageInstaller(string packageManager, ResourceLoggerService loggerService, ResourceNotificationService notificationService)
+internal class NodePackageInstaller(string packageManager, ResourceLoggerService loggerService, ResourceNotificationService notificationService)
 {
     /// <summary>
     /// Finds the Node.js resources using the specified package manager and installs the packages.

diff --git a/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/PathNormalizer.cs b/src/CommunityToolkit.Aspire.Hosting.NodeJS.Extensions/PathNormalizer.cs
@@ -1,4 +1,4 @@
-namespace CommunityToolkit.Aspire.Hosting.NodeJS.Extensions;
+namespace Aspire.Hosting.Utils;
 
 // Copied from https://github.com/dotnet/aspire/blob/50ca9fa670af5c70782dc75d2961956b06f1a403/src/Shared/PathNormalizer.cs
 internal static class PathNormalizer