diff --git a/.github/workflows/lint.yaml b/.github/workflows/lint.yaml index b1bfbf8812..db70d646f4 100644 --- a/.github/workflows/lint.yaml +++ b/.github/workflows/lint.yaml @@ -50,10 +50,18 @@ jobs: # It is used to set as required check for the branch protection rules go-lint-completed: runs-on: ubuntu-24.04 + if: always() needs: go steps: - - run: | - echo completed + - name: Check if all go lint jobs succeeded + # if jobs in the 'go' job matrix failed or were cancelled, this job will fail + # otherwise this job is marked as successful because all steps are skipped + run: exit 1 + if: >- + ${{ + contains(needs.*.result, 'failure') + || contains(needs.*.result, 'cancelled') + }} web: runs-on: ubuntu-24.04 diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml index 7dc0b57e98..b91cf1612d 100644 --- a/.github/workflows/test.yaml +++ b/.github/workflows/test.yaml @@ -58,10 +58,18 @@ jobs: # It is used to set as required check for the branch protection rules go-test-completed: runs-on: ubuntu-24.04 + if: always() needs: go steps: - - run: | - echo completed + - name: Check if all go test jobs succeeded + # if jobs in the 'go' job matrix failed or were cancelled, this job will fail + # otherwise this job is marked as successful because all steps are skipped + run: exit 1 + if: >- + ${{ + contains(needs.*.result, 'failure') + || contains(needs.*.result, 'cancelled') + }} web: runs-on: ubuntu-24.04 diff --git a/RELEASE b/RELEASE index 72f1269df7..4c70a69314 100644 --- a/RELEASE +++ b/RELEASE @@ -1,6 +1,6 @@ # Generated by `make release` command. # DO NOT EDIT. -tag: v0.54.1 +tag: v0.54.2 releaseNoteGenerator: showCommitter: false diff --git a/tool/actions-plan-preview/planpreview.go b/tool/actions-plan-preview/planpreview.go index bb7583e74a..dcffd90f81 100644 --- a/tool/actions-plan-preview/planpreview.go +++ b/tool/actions-plan-preview/planpreview.go @@ -244,7 +244,7 @@ func makeCommentBody(event *githubEvent, r *PlanPreviewResult, title string) str for _, ppr := range app.PluginPlanResults { fmt.Fprintf(&b, " - %s(%s): %s\n", ppr.PluginName, ppr.DeployTarget, ppr.PlanSummary) } - fmt.Fprint(&b, " Details:\n") + fmt.Fprint(&b, "\n Details:\n") for _, ppr := range app.PluginPlanResults { fmt.Fprintf(&b, " - %s(%s):\n", ppr.PluginName, ppr.DeployTarget) diff --git a/tool/actions-plan-preview/testdata/comment-plugin-has-failed-app.txt b/tool/actions-plan-preview/testdata/comment-plugin-has-failed-app.txt index 75ad24583e..0414b10b1a 100644 --- a/tool/actions-plan-preview/testdata/comment-plugin-has-failed-app.txt +++ b/tool/actions-plan-preview/testdata/comment-plugin-has-failed-app.txt @@ -10,6 +10,7 @@ Sync strategy: PIPELINE Plugin(s): kubernetes Summary: - kubernetes(dt-1): 2 resources will be added, 1 resource will be deleted and 5 resources will be changed + Details: - kubernetes(dt-1):
diff --git a/tool/actions-plan-preview/testdata/comment-plugin-one-success-app-multiple-plugins.txt b/tool/actions-plan-preview/testdata/comment-plugin-one-success-app-multiple-plugins.txt index 29848849d3..9f89240cb3 100644 --- a/tool/actions-plan-preview/testdata/comment-plugin-one-success-app-multiple-plugins.txt +++ b/tool/actions-plan-preview/testdata/comment-plugin-one-success-app-multiple-plugins.txt @@ -11,6 +11,7 @@ Sync strategy: PIPELINE Summary: - kubernetes(dt-1): 2 resources will be added, 1 resource will be deleted and 5 resources will be changed - terraform(dt-2): 1 resource will be added, 2 resources will be deleted and 3 resources will be changed + Details: - kubernetes(dt-1):
diff --git a/tool/actions-plan-preview/testdata/comment-plugin-one-success-app.txt b/tool/actions-plan-preview/testdata/comment-plugin-one-success-app.txt index 46ce9aee76..b7bb47f955 100644 --- a/tool/actions-plan-preview/testdata/comment-plugin-one-success-app.txt +++ b/tool/actions-plan-preview/testdata/comment-plugin-one-success-app.txt @@ -10,6 +10,7 @@ Sync strategy: QUICK_SYNC Plugin(s): kubernetes Summary: - kubernetes(dt-1): 2 resources will be added, 1 resource will be deleted and 5 resources will be changed + Details: - kubernetes(dt-1):