Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

Add example deploy scripts for reference #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hexonaut
Copy link
Contributor

@hexonaut hexonaut commented Mar 2, 2022

Helpful for reference when doing a deploy.

@hexonaut hexonaut self-assigned this Mar 2, 2022
@hexonaut hexonaut requested a review from talbaneth March 2, 2022 16:15

echo "Set permissions..."

seth send $CROPPER 'rely(address)' $MCD_PAUSE_PROXY
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still just glancing but might be good to check here that MCD_PAUSE_PROXY is not empty (also a in the other script).
Just to avoid this failing, then denying and having to redeploy.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or we can just use the chainlog to get the pause proxy addr


echo "Deploying contracts..."

CROPPER=$(dapp create Cropper)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe adding the gas limit for each call or one that covers the maximum one?
As by default it will try to run with 200K and the contract creation ones will fail.

#
# Usage: ./deploy-cropper.sh
####################################################

Copy link
Contributor

@talbaneth talbaneth Mar 3, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Subjective so just fyi -
Might be useful to add "set -x" at the top of such scripts as it echoes each command and you can see the actual seth commands performed with the actual addresses.

Problem here is that it will also echo the echo commands, so these will be printed twice.
But you can maybe remove them.
Anw feel free to ignore :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants