Added ToDo comments; Emptied constructor

microsoft · Nov 28, 2023 · 1d64ace · 1d64ace
1 parent 568eede
commit 1d64ace
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/AzureExtension/DevBox/DevBoxInstance.cs b/src/AzureExtension/DevBox/DevBoxInstance.cs
@@ -76,6 +76,7 @@ public DevBoxInstance(IDevBoxAuthService devBoxAuthService)
     // Returns a DevBox object from a JSON object
     public void FillFromJson(JsonElement item, string project, IDeveloperId? devId) => throw new NotImplementedException();
 
+    // ToDo: Use System.Uri
     private (string WebURI, string RdpURI) GetRemoteLaunchURIsAsync(string boxURI)
     {
          return (string.Empty, string.Empty);

diff --git a/src/AzureExtension/DevBox/DevBoxProvider.cs b/src/AzureExtension/DevBox/DevBoxProvider.cs
@@ -15,9 +15,8 @@ public class DevBoxProvider : IComputeSystemProvider, IDisposable
 {
     private readonly IHost _host;
 
-    public DevBoxProvider(IHost host)
+    public DevBoxProvider()
     {
-        _host = host;
     }
 
     string IComputeSystemProvider.DefaultComputeSystemProperties
@@ -50,6 +49,7 @@ public IEnumerable<IComputeSystem> GetComputeSystemsAsync(IDeveloperId? develope
         }
         else
         {
+            // ToDo: Remove throw and add to return object
             Log.Logger()?.ReportError($"Error getting systems: Rest Service not configured");
             throw new ArgumentException($"Rest Service needs to be configured.");
         }