Skip to content

Commit 8072d24

Browse files
authored
Chore: make CI output more concise (#57)
1 parent 9591fa3 commit 8072d24

File tree

3 files changed

+20
-18
lines changed

3 files changed

+20
-18
lines changed

.github/workflows/ci.yml

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,22 +11,21 @@ concurrency:
1111
cancel-in-progress: true
1212

1313
jobs:
14-
# Disable Lint
15-
16-
# Lint:
17-
# runs-on: ubuntu-latest
14+
Lint:
15+
runs-on: ubuntu-latest
1816

19-
# steps:
20-
# - uses: actions/checkout@v4
21-
# - name: Install stable Rust
22-
# uses: actions-rs/toolchain@v1
23-
# with:
24-
# toolchain: stable
25-
# components: rustfmt, clippy
17+
steps:
18+
- uses: actions/checkout@v4
19+
- name: Install stable Rust
20+
uses: actions-rs/toolchain@v1
21+
with:
22+
toolchain: stable
23+
components: rustfmt, clippy
2624

27-
# - name: Fmt
28-
# run: cargo fmt --check
25+
- name: Fmt
26+
run: cargo fmt --check
2927

28+
# TODO: enable clippy later
3029
# - name: Clippy
3130
# run: cargo clippy -- -D warnings
3231

scripts/e2e_test.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,5 +25,5 @@ cp proof.bin $REPO_ROOT/src/e2e/encoded/
2525
# Return to the root directory
2626
cd $REPO_ROOT
2727

28-
# Execute the test_zkvm_proof_verifier test
29-
RUST_LOG=info cargo test --release --lib test_zkvm_proof_verifier -- --nocapture
28+
# Execute the test_zkvm_verifier test
29+
RUST_LOG=info,p3_=warn cargo test --release --lib test_zkvm_verifier -- --nocapture

src/e2e/mod.rs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,19 @@
11
use crate::basefold_verifier::basefold::BasefoldCommitment;
22
use crate::tower_verifier::binding::IOPProverMessage;
3-
use crate::zkvm_verifier::binding::{TowerProofInput, ZKVMProofInput, ZKVMChipProofInput, E, F};
3+
use crate::zkvm_verifier::binding::{TowerProofInput, ZKVMChipProofInput, ZKVMProofInput, E, F};
44
use crate::zkvm_verifier::verifier::verify_zkvm_proof;
55

6-
use mpcs::{Basefold, BasefoldRSParams};
76
use ceno_zkvm::scheme::ZKVMProof;
87
use ceno_zkvm::structs::ZKVMVerifyingKey;
8+
use mpcs::{Basefold, BasefoldRSParams};
99

10-
use openvm_native_recursion::hints::Hintable;
1110
use openvm_circuit::arch::instructions::program::Program;
1211
use openvm_native_compiler::{
1312
asm::AsmBuilder,
1413
conversion::{convert_program, CompilerOptions},
1514
prelude::AsmCompiler,
1615
};
16+
use openvm_native_recursion::hints::Hintable;
1717

1818
pub fn parse_zkvm_proof_import(
1919
zkvm_proof: ZKVMProof<E, Basefold<E, BasefoldRSParams>>,
@@ -208,7 +208,10 @@ pub fn build_zkvm_verifier_program(
208208
builder.halt();
209209

210210
// Compile program
211+
#[cfg(feature = "bench-metrics")]
211212
let options = CompilerOptions::default().with_cycle_tracker();
213+
#[cfg(not(feature = "bench-metrics"))]
214+
let options = CompilerOptions::default();
212215
let mut compiler = AsmCompiler::new(options.word_size);
213216
compiler.build(builder.operations);
214217
let asm_code = compiler.code();

0 commit comments

Comments
 (0)