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.

Sign up for GitHub

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

Jump to bottom

infra: run tests for integrations and core packages separately #2380

Merged
merged 24 commits into from
Mar 24, 2022
Merged

infra: run tests for integrations and core packages separately #2380

merged 24 commits into from
Mar 24, 2022

Conversation

tzachbon
Copy link
Contributor

@tzachbon tzachbon commented Mar 8, 2022 •
edited
Loading

A really simple test-runner tailored to our specific needs that uses mocha test runner behind the scenes.

What we need:

  • We want to run our integrations and core packages one after the other in CI.
  • A way to run tests for selected packages from the workspace CWD.
  • Remove unused launch configurations

@tzachbon tzachbon self-assigned this Mar 9, 2022
@tzachbon tzachbon added infrastructure Git, CI or otherwise infrastructure related tests Testing related labels Mar 9, 2022
@barak007
Copy link
Collaborator

barak007 commented Mar 9, 2022

let's talk with @AviVahl before merge.

@tzachbon tzachbon requested a review from AviVahl March 13, 2022 09:09
@tzachbon tzachbon marked this pull request as ready for review March 15, 2022 09:10
@tzachbon tzachbon merged commit 3befb2e into master Mar 24, 2022
@tzachbon tzachbon deleted the tzach/run-tests-for-integrations-and-core-packages-separately branch March 24, 2022 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@barak007 barak007 barak007 approved these changes

@AviVahl AviVahl Awaiting requested review from AviVahl

Assignees

@tzachbon tzachbon

Labels
infrastructure Git, CI or otherwise infrastructure related tests Testing related
Projects
Stylable
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tzachbon @barak007