Skip to content

Commit eee7429

Browse files
authored
ci: Make prover e2e test mandatory (#3305)
## What ❔ Make prover e2e test mandatory for merging the PRs Fix prover e2e test ## Why ❔ <!-- Why are these changes done? What goal do they contribute to? What are the principles behind them? --> <!-- Example: PR templates ensure PR reviewers, observers, and future iterators are in context about the evolution of repos. --> ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [ ] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [ ] Tests for the changes have been added / updated. - [ ] Documentation comments have been added / updated. - [ ] Code has been formatted via `zkstack dev fmt` and `zkstack dev lint`. fix ZKD-2081
1 parent 7c93c47 commit eee7429

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/ci-prover-e2e.yml

+2-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,8 @@ jobs:
5050
--base-token-price-nominator 1 \
5151
--base-token-price-denominator 1 \
5252
--set-as-default true \
53-
--ignore-prerequisites
53+
--ignore-prerequisites \
54+
--evm-emulator false
5455
5556
ci_run zkstack ecosystem init --dev --verbose
5657
ci_run zkstack prover init --dev --verbose

.github/workflows/ci.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ jobs:
186186
name: Github Status Check
187187
runs-on: ubuntu-latest
188188
if: always() && !cancelled()
189-
needs: [ ci-for-core-lint, ci-for-common, ci-for-core, ci-for-prover, ci-for-docs, build-core-images, build-contract-verifier, build-prover-images ]
189+
needs: [ ci-for-core-lint, ci-for-common, ci-for-core, ci-for-prover, ci-for-docs, build-core-images, build-contract-verifier, build-prover-images, e2e-for-prover ]
190190
steps:
191191
- name: Status
192192
run: |

0 commit comments

Comments
 (0)