Skip to content

Commit 21c8b80

Browse files
petrochukanpetroc
andauthored
Expose fail-on-analysis-error flag (#307)
Co-authored-by: Andrew Petrochuk <[email protected]>
1 parent bfc8b50 commit 21c8b80

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

check-solution/action.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,11 @@ inputs:
4848
description: The name of the artifact folder of which the PowerApps Checker logs will be uploaded. Default value is \'CheckSolutionLogs\'
4949
required: false
5050

51+
fail-on-analysis-error:
52+
description: Fail task if the Power Apps Checker analysis is returned as Failed or FinishedWithErrors. Default value is true
53+
required: false
54+
default: 'true'
55+
5156
runs:
5257
using: 'node16'
5358
main: '../dist/actions/check-solution/index.js'

src/actions/check-solution/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,14 @@ import { runnerParameters } from '../../lib/runnerParameters';
2121
geoInstance: parameterMap["geo"],
2222
ruleLevelOverride: parameterMap["rule-level-override"],
2323
artifactStoreName: parameterMap["checker-logs-artifact-name"],
24+
failOnAnalysisError: parameterMap["fail-on-analysis-error"],
2425
fileLocation: createEntry("fileLocation", "localFiles"),
2526
solutionUrl: createEntry("solutionUrl", ""),
2627
useDefaultPAEndpoint: createEntry("useDefaultPAEndpoint", true),
2728
customPAEndpoint: createEntry("customPAEndpoint", "https://unitedstates.api.advisor.powerapps.com/"),
2829
ruleSet: createEntry("ruleSet", "Solution Checker"),
2930
errorLevel: createEntry("errorLevel", "HighIssueCount"),
3031
errorThreshold: createEntry("errorThreshold", "0"),
31-
failOnAnalysisError: createEntry("failOnAnalysisError", true),
3232
filesExcluded: createEntry("filesExcluded", "")
3333
}, runnerParameters, new ActionsHost('PowerAppsChecker'));
3434
core.endGroup();

src/test/checkSolution.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,14 +42,14 @@ describe("check solution test", () => {
4242
geoInstance: { name: 'geo', required: false, defaultValue: undefined },
4343
ruleLevelOverride: { name: 'rule-level-override', required: false, defaultValue: undefined },
4444
artifactStoreName: { name: 'checker-logs-artifact-name', required: false, defaultValue: undefined },
45+
failOnAnalysisError: { name: "fail-on-analysis-error", required: false, defaultValue: 'true' },
4546
fileLocation: { name: "fileLocation", required: false, defaultValue: "localFiles" },
4647
solutionUrl: { name: "solutionUrl", required: false, defaultValue: "" },
4748
useDefaultPAEndpoint: { name: "useDefaultPAEndpoint", required: false, defaultValue: true },
4849
customPAEndpoint: { name: "customPAEndpoint", required: false, defaultValue: "https://unitedstates.api.advisor.powerapps.com/" },
4950
ruleSet: { name: "ruleSet", required: false, defaultValue: "Solution Checker" },
5051
errorLevel: { name: "errorLevel", required: false, defaultValue: "HighIssueCount" },
5152
errorThreshold: { name: "errorThreshold", required: false, defaultValue: "0" },
52-
failOnAnalysisError: { name: "failOnAnalysisError", required: false, defaultValue: true },
5353
filesExcluded: { name: "filesExcluded", required: false, defaultValue: ""}
5454
}, runnerParameters, new ActionsHost('PowerAppsChecker'));
5555
});

0 commit comments

Comments
 (0)