Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: attempt to parallelize acceptance tests #4415

Closed
wants to merge 11 commits into from

Conversation

asaf92
Copy link
Contributor

@asaf92 asaf92 commented Feb 15, 2023

Attempting to run CircleCI

@asaf92 asaf92 requested a review from a team as a code owner February 15, 2023 09:22
@@ -333,6 +334,7 @@ jobs:
type: string
executor: win/default
working_directory: C:\Users\circleci\snyk
parallelism: 4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is that the number of cores on the worker?

- run:
name: Installing SDKs
command: choco install .circleci/chocolatey.config --no-progress
- save_cache:
name: Saving Chocolatey cache
key: chocolatey-cache-v2-{{ arch }}-{{ checksum ".circleci/chocolatey.config" }}
key: chocolatey-cache-v20230216-{{ arch }}-{{ checksum ".circleci/chocolatey.config" }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the different cache-key?

@@ -85,7 +85,7 @@ commands:
- restore_cache:
name: Restoring npm cache
keys:
- npm-cache-v2-{{ arch }}-node<< parameters.node_version >>-npm<< parameters.npm_version >>-{{ checksum "package-lock.json" }}
- npm-cache-v3-{{ arch }}-node<< parameters.node_version >>-npm<< parameters.npm_version >>-{{ checksum "package-lock.json" }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the key change?

bastiandoetsch
bastiandoetsch previously approved these changes Feb 22, 2023
@asaf92 asaf92 force-pushed the test/optimize-jest-tests-attempt branch 2 times, most recently from e79ffa8 to 058ece8 Compare February 23, 2023 13:16
@asaf92 asaf92 force-pushed the test/optimize-jest-tests-attempt branch from 058ece8 to 75b1b89 Compare February 26, 2023 13:13
@j-luong
Copy link
Contributor

j-luong commented Jan 5, 2024

Closing stale PR

@j-luong j-luong closed this Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants