From 3f486a9b99b025545e4982c34294fca8837cd60c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Omar=20L=C3=B3pez?= Date: Thu, 3 Nov 2022 13:29:14 -0700 Subject: [PATCH] Revert "Fix (#5293)" (#5359) This reverts commit f934dc6f4fc1b8e04b6281c31881fb8ec5028ffc. --- .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 cf2f26acca..9e702440e8 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: # So we can test on forks + pull_request_target: # So we can test on forks branches: - main paths: diff --git a/.github/workflows/check-types.yml b/.github/workflows/check-types.yml index ec8ed97585..bb7a348182 100644 --- a/.github/workflows/check-types.yml +++ b/.github/workflows/check-types.yml @@ -1,6 +1,6 @@ name: Check types on: - pull_request: + pull_request_target: branches: - main paths: diff --git a/.github/workflows/e2e-app-store.yml b/.github/workflows/e2e-app-store.yml index 293521ecc2..ebb0edf01f 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: # So we can test on forks + pull_request_target: # 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 f95d5a7f7e..ff3202776d 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: # So we can test on forks + pull_request_target: # 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 fe6cf07cae..405ad6f381 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: # So we can test on forks + pull_request_target: # So we can test on forks branches: - main paths: diff --git a/.github/workflows/labeler.yml b/.github/workflows/labeler.yml index f7599b5167..5c9910a83f 100644 --- a/.github/workflows/labeler.yml +++ b/.github/workflows/labeler.yml @@ -1,6 +1,6 @@ name: "Pull Request Labeler" on: - - pull_request + - pull_request_target 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 b444036958..10ed6b68ef 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -1,6 +1,6 @@ name: Lint on: - pull_request: + pull_request_target: branches: - main paths: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 9ea3a2e3e6..f8e2f57d13 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: # So we can test on forks + pull_request_target: # So we can test on forks branches: - main paths: