Solution: LP-0002 Private M-of-N Multisig - #92
Conversation
✅ Validation passedA reviewer will assess against the prize criteria. Automated check. See solution template and TERMS. |
|
Preliminary checking your submission I notice that transactions are not visible on Can you, please, re-submit them? |
|
Hi @Tranquil-Flow ! Really value your submission and thanks for addressing previous review. However I have comments for this submission that I would appreciate you addressing:
|
|
Hi @weboko, thanks for the review. I’ve now addressed both points:
Updated submission: Tranquil-Flow@5a3dd3b Green E2E CI: https://github.com/Tranquil-Flow/lp-0002-private-multisig/actions/runs/31475962918 Could you please take another look and reopen the PR if appropriate? Thanks! |
LP-0002 resubmission after addressing reviewer feedback from PR #68.
Reviewer feedback addressed:
Deploy on real LEZ testnet — The wrapper program is now deployed and executed on the public LEZ testnet (https://testnet.lez.logos.co/). Deploy tx
82516880...in block 39547, execute txcb8bfd5...in block 39548, signed by funded testnet account6iArKUXxhUJqS7kCaPNhwMWt3ro71PDyBj7jwAyE2VQV. Evidence recorded insubmission/TESTNET_EVIDENCE.json.Document per-operation CU/gas —
submission/LEZ_COST_BENCHMARKS.jsonrecords the explicit CU-unavailable rationale (LEZ tooling does not expose stable per-tx CU counters) along with payload/account/receipt/block metrics.demo.sh end-to-end against live sequencer with RISC0_DEV_MODE=0 — The heavy-lane path (
scripts/demo-heavy-lane.sh) runs the real end-to-end flow against the LEZ testnet withRISC0_DEV_MODE=0proof artifacts.e2e-against-sequencer integration tests in CI — CI now includes
.github/workflows/ci.ymlalongside.gitlab-ci.yml, both green on the default branch.Additional improvements since PR #68:
.lgxpackage with alice/bob install evidencePublic repo: https://github.com/Tranquil-Flow/lp-0002-private-multisig
Thank you for the detailed feedback on #68. All points have been addressed.