-
Notifications
You must be signed in to change notification settings - Fork 0
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
merge to main #2
base: main
Are you sure you want to change the base?
Conversation
v1.9.2
Feat/deploy script
updated deployment scripts
Feat/use precomputed tokens
remappings.txt
Outdated
@@ -0,0 +1,19 @@ | |||
@openzeppelin-upgrades-v4.9.0/=lib/eigenlayer-contracts/lib/openzeppelin-contracts-upgradeable-v4.9.0/ |
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.
this remappings file is insanely complex and contributed to all the issues plagueing the wrong import of the wrong OZ library.
I have no clue what's going on here
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.
Fixed by myself and pushed directly to dev
@@ -0,0 +1,89 @@ | |||
# Default values |
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.
unnecessarily complex Makefile
we only need to deploy and test
Let's keep everything minimal and simple, no extra boilerplate
don't clean this one now, but going forward.
@@ -0,0 +1,62 @@ | |||
#!/bin/bash |
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.
We have no use going forward for boilerplate custom scripts converting CSV to json.
Ask the team to provide JSON, we have no reason to have all this code produced.
script/DeployEigenAirdrop.s.sol
Outdated
import { ProxyAdmin } from "@openzeppelin-v5.0.2/proxy/transparent/ProxyAdmin.sol"; | ||
import { TransparentUpgradeableProxy } from | ||
"@openzeppelin-v5.0.2/proxy/transparent/TransparentUpgradeableProxy.sol"; | ||
import { Address } from "@openzeppelin/contracts/utils/Address.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.
what version of OZ is this?
script/DeployEigenAirdrop.s.sol
Outdated
import { TransparentUpgradeableProxy } from | ||
"@openzeppelin-v5.0.2/proxy/transparent/TransparentUpgradeableProxy.sol"; | ||
import { Address } from "@openzeppelin/contracts/utils/Address.sol"; | ||
import { Strings } from "@openzeppelin/contracts/utils/Strings.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.
what version of OZ is this?
No description provided.