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
The test suite is really slow when run via Wasm, so I think we should probably only run the test suite via Wasm when a PR hits the merge queue. We can also add a workflow_dispatch trigger so that the test suite can be run manually on any given PR.workflow_dispatch is a GitHub thing, and we're using Buildkite. Instead, we can use a special label on a PR when we want to run the Wasm tests prior to hitting the merge queue.
(There will probably be a very short time window during which you can label the PR before Buildkite starts its evaluation, so it will probably be necessary to rebuild the Wasm build after adding the label in most cases. I don't think this will be a big deal, though.)
The text was updated successfully, but these errors were encountered:
It might also be a good idea to add a label that skips the Wasm builds entirely. This would be useful for PRs that, say, make a doc change that doesn't affect the Wasm build at all. Our Nix builds will detect that no changes have been made to the Nix derivations and skip those builds, but our Wasm builds don't cache artifacts between runs.
The test suite is really slow when run via Wasm, so I think we should probably only run the test suite via Wasm when a PR hits the merge queue.
We can also add aworkflow_dispatch
trigger so that the test suite can be run manually on any given PR.workflow_dispatch
is a GitHub thing, and we're using Buildkite. Instead, we can use a special label on a PR when we want to run the Wasm tests prior to hitting the merge queue.(There will probably be a very short time window during which you can label the PR before Buildkite starts its evaluation, so it will probably be necessary to rebuild the Wasm build after adding the label in most cases. I don't think this will be a big deal, though.)
The text was updated successfully, but these errors were encountered: