switch from list to raw

Azure · Aug 9, 2024 · bd64dc5 · bd64dc5
1 parent c4989f1
commit bd64dc5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/action.yml b/action.yml
@@ -24,7 +24,7 @@ inputs:
     required: false
     default: false
   no-job-summary:
-    description: 'Do not contribute to a job summary displayed as a job result.'
+    description: 'Do not provide a job summary from this step.'
     required: false
     default: false
 runs:

diff --git a/lib/main.js b/lib/main.js
diff --git a/src/AzureSqlAction.ts b/src/AzureSqlAction.ts
@@ -133,17 +133,15 @@ export default class AzureSqlAction {
                     const lines = buildOutput.split(/\r?\n/);
                     let warnings = lines.filter(line => (line.includes('Build warning') || line.includes('StaticCodeAnalysis warning')));
                     warnings = [...new Set(warnings)];
-                    let formattedWarnings: string[] = [];
+
                     warnings.forEach(warning => {
                         // remove [project path] from the end of the line
                         warning = warning.lastIndexOf('[') > 0 ? warning.substring(0, warning.lastIndexOf('[')-1) : warning;
 
                         // move the file info from the beginning of the line to the end
-                        warning = '**'+warning.substring(warning.indexOf(':')+2) + '** ' + warning.substring(0, warning.indexOf(':')); 
-                        formattedWarnings.push(warning);
+                        warning = '- **'+warning.substring(warning.indexOf(':')+2) + '** ' + warning.substring(0, warning.indexOf(':')); 
+                        core.summary.addRaw(warning, true);
                     });
-
-                    core.summary.addList(formattedWarnings, false);
                     core.summary.addRaw('See the full build log for more details.');
                 } else { // no build warnings
                     core.summary.addHeading(':white_check_mark: SQL project build succeeded.');
@@ -152,7 +150,7 @@ export default class AzureSqlAction {
                 core.summary.addHeading(':x: Build failed.');
             }
         } catch (err) {
-            core.warning(`Error parsing build output for job summary: ${err}`);
+            core.notice(`Error parsing build output for job summary: ${err}`);
         }
 
         const dacpacPath = path.join(outputDir, projectName + Constants.dacpacExtension);