Skip to content

Commit

Permalink
Avoid having CI build things twice
Browse files Browse the repository at this point in the history
This reverts the change in 42751dd. Something with the workspace behavior of `cargo build -p amm` causes builds of things that are outside the dependency path of amm, but this can be avoided by just entering the crate directory.
  • Loading branch information
samtay committed Feb 1, 2024
1 parent 99ad7b1 commit 486cda6
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 2,264 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -150,4 +150,5 @@ jobs:
- name: Add Rust wasm32-unknown-emscripten target
run: rustup target add wasm32-unknown-emscripten
- name: Build AMM target for Emscripten
run: source emsdk/emsdk/emsdk_env.sh; cargo build --bin amm --target wasm32-unknown-emscripten --release
run: source ../../emsdk/emsdk/emsdk_env.sh; cargo build --target wasm32-unknown-emscripten --release
working-directory: ./examples/amm
7 changes: 7 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 1 addition & 6 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,7 @@ members = [
"sunscreen_compiler_common",
"sunscreen_zkp_backend",
]
exclude = [
"mdBook",
"rust-playground",
"examples/allowlist_zkp",
"examples/private_tx_linkedproof",
]
exclude = ["mdBook", "rust-playground", "examples/allowlist_zkp"]

[profile.release]
split-debuginfo = "packed"
Expand Down
Loading

0 comments on commit 486cda6

Please sign in to comment.