Merge branch 'main' of https://github.com/TheCloudTheory/arm-estimator

TheCloudTheory · Aug 30, 2024 · fd141db · fd141db
2 parents 9508897 + 64607cb
commit fd141db
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/tests-scheduled.yml b/.github/workflows/tests-scheduled.yml
@@ -20,7 +20,7 @@ jobs:
     steps:
     - uses: actions/checkout@v3
     - name: Azure Login
-      uses: Azure/login@v1.4.6
+      uses: Azure/login@v2.1.0
       with:
         creds: ${{ secrets.SP_CREDS }}
     - name: HashiCorp - Setup Terraform

diff --git a/ace/Compilation/BicepCompiler.cs b/ace/Compilation/BicepCompiler.cs
@@ -41,7 +41,7 @@ ILogger<Program> logger
             {
                 // First compilation may not work if Azure CLI is not installed directly,
                 // try to use Bicep CLI instead
-                this.logger.AddEstimatorMessage("Compilation failed, attempting to compile Bicep file using Azure CLI.");
+                this.logger.AddEstimatorMessage("Compilation failed, attempting to compile Bicep file using Bicep CLI.");
                 CompileBicepWith("bicep", $"build {templateFile} --stdout", token, logger, out template);
             }
 

diff --git a/ace/Program.cs b/ace/Program.cs
@@ -297,7 +297,7 @@ private static async Task<ApplicationResult> Estimate(FileInfo templateFile,
 
             if (whatIfData != null && whatIfData.status == "Failed")
             {
-                var error = "An error happened when performing WhatIf operation.";
+                var error = $"An error happened when performing WhatIf operation with status {whatIfData.status}.";
                 logger.LogError("{error}", error);
 
                 if (whatIfData.error != null)