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

feat(parallel): support runParallel in checks and groups [sc-18210] #900

Merged

Conversation

shiini2
Copy link
Contributor

@shiini2 shiini2 commented Dec 1, 2023

I hereby confirm that I followed the code guidelines found at engineering guidelines

Affected Components

  • CLI
  • Create CLI
  • Test
  • Docs
  • Examples
  • Other

Notes for the Reviewer

Add runParallel to check and group constructs to allow simultaneous check run scheduling on multi-locations.

If set to false, the check runs will be scheduled in the different locations in a round-robin fashion.

Requires this BE PR for testing.

Copy link

This pull request has been linked to Shortcut Story #18210: Add runParallel to CLI checks construct.

@ndom91
Copy link
Contributor

ndom91 commented Dec 4, 2023

The currently failing tests are due to them using an outdated version of the BE, right? (outdated meaning they don't have the changes in the BE PR that is).

Because otherwise, LGTM 👍

Worked locally with both PRs checked out

@shiini2
Copy link
Contributor Author

shiini2 commented Dec 4, 2023

The currently failing tests are due to them using an outdated version of the BE, right? (outdated meaning they don't have the changes in the BE PR that is).

Yes exactly, once the schema update is deployed, the tests here should pass 🤞

@maxigimenez maxigimenez added the build Issue regarding building and packaging label Dec 14, 2023
Copy link

🎉 Experimental release successfully published on npm

npm install checkly@0.0.0-pr.900.a4cade6

@clample clample merged commit 4c52015 into main Dec 15, 2023
3 checks passed
@clample clample deleted the manelfourati/sc-18210/add-runparallel-to-cli-checks-construct branch December 15, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issue regarding building and packaging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants