Skip to content

Commit 3f49f4e

Browse files
authored
Merge pull request #9144 from nadavMiz/remove-push-directive
remove push directive so we will not run tests on merge
2 parents 8762e9b + 849ea70 commit 3f49f4e

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

.github/workflows/Validate-package-lock.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: Validate package-lock.json Tests
2-
on: [push, pull_request]
2+
on: [pull_request]
33

44
jobs:
55
run-package-lock-validation:

.github/workflows/jest-unit-tests.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: Jest Unit Tests
2-
on: [push, pull_request, workflow_dispatch]
2+
on: [pull_request, workflow_dispatch]
33

44
jobs:
55
run-jest-unit-tests:

.github/workflows/run-pr-tests.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
name: Run PR Tests
2-
on: [push, pull_request, workflow_dispatch]
2+
on: [pull_request, workflow_dispatch]
33
concurrency:
44
group: ${{ github.workflow }}-${{ github.ref }}
55
cancel-in-progress: true
@@ -66,7 +66,7 @@ jobs:
6666
#on pull request, use target branch
6767
BRANCH=${{github.base_ref}}
6868
else
69-
#on push, use the pushed branch
69+
#on dispach, use the current branch
7070
BRANCH=${{github.ref_name}}
7171
fi
7272
BUILDER_TAGS="${DOCKER_BUILDER_IMAGE}:${BRANCH}-"

0 commit comments

Comments
 (0)