-
Notifications
You must be signed in to change notification settings - Fork 154
feat: add template for funding from safe #329
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 8000 to your account
Conversation
✅ Heimdall Review Status
|
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.
Functionally looks good to me! Mostly just nits from my end. I also think we should update the root README accordingly
Makefile
Outdated
git remote add origin https://github.com/ethereum-optimism/optimism.git; \ | ||
git fetch --depth=1 origin $(OP_COMMIT); \ | ||
git reset --hard FETCH_HEAD | ||
@if grep -q "^OP_COMMIT=" .env 2>/dev/null; then \ |
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 task won't build for me without an OP_COMMIT. I think it's because the base contracts use OP utility files under the hood
This reverts commit 599706e.
Add a template to easily create tasks for funding new facilitator ledgers.