-
Notifications
You must be signed in to change notification settings - Fork 338
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
feat: wip refactor of deploy scripts #686
base: dev
Are you sure you want to change the base?
Conversation
based, glad to collab |
release/README.md
Outdated
``` | ||
|
||
* Generates a script file (`preprod-<GIT_COMMIT_HEAD>.s.sol`) that automatically loads the config for the `preprod` environment | ||
* `make release preprod --test preprod-<GIT_COMMIT_HEAD>.s.sol`: |
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.
I like to use the specific release tag vs commit hash for more clarity but like the idea nonetheless
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.
i like that but i'm kinda forseeing a scenario where we don't have a release tagged yet
e870331
to
df090b9
Compare
|
1d03f5b
to
76b6760
Compare
9f6543b
to
3cd7153
Compare
* chore: remove unused release folder
3cd7153
to
fd8c2e1
Compare
Collecting vibes here: