Skip to content

Commit d854681

Browse files
fix: don't require GSSA run on docs only changes (#477)
### Description GSSA incorrectly required on PRs with docs only changes. ### Checklist - [ ] `README.md` has been updated or is not required - [ ] push trigger tests - [ ] manual release test - [ ] automated releases test - [ ] pull request trigger tests - [ ] schedule trigger tests - [ ] workflow errors/warnings reviewed and addressed ### Testing done splunk/splunk-add-on-for-okta-identity-cloud#420
1 parent cb8f99e commit d854681

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/reusable-build-test-release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2936,7 +2936,7 @@ jobs:
29362936
run: |
29372937
# GS Scorecard: must have run for PRs to main (result doesn't matter, but skipped = blocked)
29382938
GS_RESULT=$(echo "$NEEDS" | jq -r '.["run-gs-scorecard"].result')
2939-
if [[ "${{ github.event_name }}" == "pull_request" ]] && [[ "${{ github.base_ref }}" == "main" ]] && [[ "$GS_RESULT" == "skipped" ]]; then
2939+
if [[ "${{ github.event_name }}" == "pull_request" ]] && [[ "${{ github.base_ref }}" == "main" ]] && [[ "$GS_RESULT" == "skipped" ]] && [[ "${{ needs.check-docs-changes.outputs.docs-only }}" != "true" ]]; then
29402940
echo "::error::GS Scorecard is required for merging PRs to main. To unblock merging please add the 'execute_gs_scorecard' label and re-run the workflow."
29412941
echo "## GS Scorecard Required" >> "$GITHUB_STEP_SUMMARY"
29422942
echo "Add the \`execute_gs_scorecard\` label to this PR and re-run the workflow. GS Scorecard must complete before merging to main (result does not need to pass)." >> "$GITHUB_STEP_SUMMARY"

0 commit comments

Comments
 (0)