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

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