Skip to content

Commit

Permalink
ci: remove test-executor
Browse files Browse the repository at this point in the history
This is a copy of action-executor with a different `resource_class`. This is a bit redundant since it is overridden in most of the cases.

(cherry picked from commit ade007b)
  • Loading branch information
alan-agius4 committed Sep 27, 2022
1 parent 2023138 commit 14ac8c6
Showing 1 changed file with 4 additions and 13 deletions.
17 changes: 4 additions & 13 deletions .circleci/dynamic_config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,16 +63,6 @@ executors:
working_directory: ~/ng
resource_class: small

test-executor:
parameters:
nodeversion:
type: string
default: *default_nodeversion
docker:
- image: cimg/node:<< parameters.nodeversion >>
working_directory: ~/ng
resource_class: large

windows-executor:
# Same as https://circleci.com/orbs/registry/orb/circleci/windows, but named.
working_directory: ~/ng
Expand Down Expand Up @@ -213,9 +203,10 @@ jobs:
enum: *all_e2e_subsets
default: 'npm'
executor:
name: test-executor
name: action-executor
nodeversion: << parameters.nodeversion >>
parallelism: 8
resource_class: large
steps:
- custom_attach_workspace
- browser-tools/install-chrome
Expand Down Expand Up @@ -249,7 +240,7 @@ jobs:

test-browsers:
executor:
name: test-executor
name: action-executor
resource_class: medium
steps:
- custom_attach_workspace
Expand Down Expand Up @@ -285,7 +276,7 @@ jobs:
- run: yarn bazel build //tests/legacy-cli/...

unit-test:
executor: test-executor
executor: action-executor
resource_class: xlarge
parameters:
nodeversion:
Expand Down

0 comments on commit 14ac8c6

Please sign in to comment.