You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Context and scope
The separated e2e test suites introduced in #592 allow each suite to be run simultaneously and in isolation. However, the validator-manager suite in particular is long running, and it is unnecessary to run those tests if, say, a PR only changes the teleporter contracts.
Discussion and alternatives
We should separate the e2e jobs in the test workflow such that each suite only runs if files in the corresponding subdirectory of contracts are changed. We should still run all tests on merging to main.
The [paths](https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore) syntax can be used to define these filters at the workflow level. Something like https://github.com/tj-actions/changed-files?tab=readme-ov-file can be used to do this at the job level.
The text was updated successfully, but these errors were encountered:
Context and scope
The separated e2e test suites introduced in #592 allow each suite to be run simultaneously and in isolation. However, the
validator-manager
suite in particular is long running, and it is unnecessary to run those tests if, say, a PR only changes theteleporter
contracts.Discussion and alternatives
We should separate the e2e jobs in the test workflow such that each suite only runs if files in the corresponding subdirectory of
contracts
are changed. We should still run all tests on merging to main.The
[paths](https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore)
syntax can be used to define these filters at the workflow level. Something like https://github.com/tj-actions/changed-files?tab=readme-ov-file can be used to do this at the job level.The text was updated successfully, but these errors were encountered: