-
Notifications
You must be signed in to change notification settings - Fork 3
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
ZK-559: Remove "/crypto" package from shielder-sdk #81
Changes from 29 commits
22d941a
fee1b6a
e9ef3f1
1d8b114
1cf65dc
473b8cf
c2cf81f
29177ca
5e81347
936d806
471a420
0abb655
3973738
b13140b
78ae484
3e7f04e
e04a43a
7926432
b33159e
0b011d0
9a37573
2fd71b8
298f56f
9335dc7
170ea2e
44f396b
71b060b
2b16902
d3b7bbe
bd3eaf3
77cc8fb
ec3956a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,7 +11,6 @@ concurrency: | |
cancel-in-progress: true | ||
|
||
jobs: | ||
|
||
e2e-tests: | ||
name: Run E2E tests on testnet | ||
runs-on: ubuntu-22.04 | ||
|
@@ -67,39 +66,6 @@ jobs: | |
shell: bash | ||
run: make compile-contracts | ||
|
||
- name: Install wasm-pack | ||
run: curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh | ||
|
||
- name: Build shielder-wasm | ||
run: cd crates/shielder-wasm && make all | ||
|
||
- name: Build Rust-TS conversions binary | ||
run: cargo build --manifest-path crates/test-ts-conversions/Cargo.toml | ||
|
||
- uses: pnpm/action-setup@v4 | ||
with: | ||
version: latest | ||
|
||
- name: Install dependencies (shielder-sdk) | ||
run: pnpm install --frozen-lockfile | ||
working-directory: ts/shielder-sdk | ||
|
||
- name: Build shielder-sdk | ||
run: pnpm build | ||
working-directory: ts/shielder-sdk | ||
|
||
- name: Install dependencies (shielder-sdk-tests) | ||
pmikolajczyk41 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
run: pnpm install --frozen-lockfile | ||
working-directory: ts/shielder-sdk-tests | ||
|
||
- name: Install Playwright dependencies (shielder-sdk-tests) | ||
run: pnpm postinstall | ||
working-directory: ts/shielder-sdk-tests | ||
|
||
- name: Build shielder-sdk-tests | ||
run: pnpm build | ||
working-directory: ts/shielder-sdk-tests | ||
|
||
- name: Run e2e tooling tests | ||
env: | ||
DEPLOYER_PRIVATE_KEY: ${{ secrets.CI_TESTNET_DEPLOYER_PRIVATE_KEY }} | ||
|
@@ -119,7 +85,6 @@ jobs: | |
NO_FORMATTING=true TESTNET=true ./tooling-e2e-tests/full_scenario.sh | ||
NO_FORMATTING=true TESTNET=true ./tooling-e2e-tests/recovery_scenario.sh | ||
NO_FORMATTING=true TESTNET=true ./tooling-e2e-tests/many_actors.sh | ||
NO_FORMATTING=true TESTNET=true ./tooling-e2e-tests/ts_sdk_tests.sh | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider removing There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. not removing it now, as it would be refactored soon. tests are disabled for CI purposes |
||
|
||
slack-notification: | ||
name: Slack notification | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
dist/ | ||
dist/ | ||
dist-vite/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we removing the workflow entirely? Playwright is needed for testing the smaller
shielder-sdk-crypto-wasm
package.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can add it later