-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update cast-on-tenderly script #388
Conversation
The updated script is ready for review and can be tested with the last spell address via |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of nits regarding naming for discussion, but I'm not feeling that strong about it. Feel free to keep it as it is.
At this point, it becomes a bit subjective unless we have some fairly comprehensive style guide in place, which we don't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm :)
Closes #389
This PR updates cast-on-tenderly script to:
cast-on-tenderly
outdated dependencies