From f934dc6f4fc1b8e04b6281c31881fb8ec5028ffc Mon Sep 17 00:00:00 2001 From: Hariom Balhara Date: Mon, 31 Oct 2022 19:28:47 +0530 Subject: [PATCH] Fix (#5293) Co-authored-by: Alex van Andel --- .github/workflows/check-if-ui-has-changed.yml | 2 +- .github/workflows/check-types.yml | 2 +- .github/workflows/e2e-app-store.yml | 2 +- .github/workflows/e2e-embed.yml | 2 +- .github/workflows/e2e.yml | 2 +- .github/workflows/labeler.yml | 2 +- .github/workflows/lint.yml | 2 +- .github/workflows/test.yml | 2 +- 8 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/check-if-ui-has-changed.yml b/.github/workflows/check-if-ui-has-changed.yml index 9e702440e8..cf2f26acca 100644 --- a/.github/workflows/check-if-ui-has-changed.yml +++ b/.github/workflows/check-if-ui-has-changed.yml @@ -5,7 +5,7 @@ name: 'Chromatic' # Event for the workflow on: - pull_request_target: # So we can test on forks + pull_request: # So we can test on forks branches: - main paths: diff --git a/.github/workflows/check-types.yml b/.github/workflows/check-types.yml index bb7a348182..ec8ed97585 100644 --- a/.github/workflows/check-types.yml +++ b/.github/workflows/check-types.yml @@ -1,6 +1,6 @@ name: Check types on: - pull_request_target: + pull_request: branches: - main paths: diff --git a/.github/workflows/e2e-app-store.yml b/.github/workflows/e2e-app-store.yml index ebb0edf01f..293521ecc2 100644 --- a/.github/workflows/e2e-app-store.yml +++ b/.github/workflows/e2e-app-store.yml @@ -2,7 +2,7 @@ name: E2E App-Store Apps on: push: branches: [fixes/e2e-consolidation] # TODO: Remove this after merged in main - pull_request_target: # So we can test on forks + pull_request: # So we can test on forks branches: - main paths-ignore: diff --git a/.github/workflows/e2e-embed.yml b/.github/workflows/e2e-embed.yml index ff3202776d..f95d5a7f7e 100644 --- a/.github/workflows/e2e-embed.yml +++ b/.github/workflows/e2e-embed.yml @@ -1,6 +1,6 @@ name: E2E Embed tests and booking flow(for non-embed as well) on: - pull_request_target: # So we can test on forks + pull_request: # So we can test on forks branches: - main # Embed e2e - tests verify booking flow which is applicable to non-embed case also. So, don't ignore apps/web changes. diff --git a/.github/workflows/e2e.yml b/.github/workflows/e2e.yml index 405ad6f381..fe6cf07cae 100644 --- a/.github/workflows/e2e.yml +++ b/.github/workflows/e2e.yml @@ -2,7 +2,7 @@ name: E2E test on: push: branches: [fixes/e2e-consolidation] # TODO: Remove this after merged in main - pull_request_target: # So we can test on forks + pull_request: # So we can test on forks branches: - main paths: diff --git a/.github/workflows/labeler.yml b/.github/workflows/labeler.yml index 5c9910a83f..f7599b5167 100644 --- a/.github/workflows/labeler.yml +++ b/.github/workflows/labeler.yml @@ -1,6 +1,6 @@ name: "Pull Request Labeler" on: - - pull_request_target + - pull_request concurrency: group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} cancel-in-progress: true diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 10ed6b68ef..b444036958 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -1,6 +1,6 @@ name: Lint on: - pull_request_target: + pull_request: branches: - main paths: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index f8e2f57d13..9ea3a2e3e6 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -2,7 +2,7 @@ name: Unit tests on: push: branches: [fixes/e2e-consolidation] # TODO: Remove this after merged in main - pull_request_target: # So we can test on forks + pull_request: # So we can test on forks branches: - main paths: