EspressoSystems / espresso-sequencer

86 stars 56 forks source link

combined the prover proposal scripts #1585

Closed alysiahuggins closed 1 week ago

alysiahuggins commented 4 weeks ago

Closes #1502

This PR:

There was previously a separate script for setting the prover and disabling the prover on the light client contract. This combines the two into one script.

Key places to review:

The command line arguments required for each proposal and the fact that the setProver depends on the env variable, APPROVED_PROVER_ADDRESS for setting the prover. This hasn't changed from the previous implementation but worth mentioning.

How to test this PR:

See the read me - it was tested this way in #1328 as well.

Things tested

Manually tested the disableProver and setProver workflow.

See PR https://github.com/EspressoSystems/espresso-sequencer/pull/1454