-
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
chore: v1.0.0 deployment addresses #175
Conversation
README.md
Outdated
|
||
Gnosis Mainnet: | ||
- BCoWFactory: (0xaD0447be7BDC80cf2e6DA20B13599E5dc859b667)[https://etherscan.io/address/0xaD0447be7BDC80cf2e6DA20B13599E5dc859b667] |
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.
address is for a BCoWFactory
in gnosis chain but link is on etherscan (where there's a previously deployed BFactory
)
README.md
Outdated
@@ -60,8 +60,14 @@ yarn test # run the tests | |||
|
|||
# Deployments | |||
Ethereum Mainnet: | |||
- BCoWFactory: (0x21Cd97D70f8475DF3d62917880aF9f41D9a9dCeF)[https://etherscan.io/address/0x21Cd97D70f8475DF3d62917880aF9f41D9a9dCeF#code] | |||
- BCoWFactory: (0x5AC134DAC7070eFeE8b1C5e3fD0B353922ceD843)[https://etherscan.io/address/0x5AC134DAC7070eFeE8b1C5e3fD0B353922ceD843] |
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.
markdown links are backwards: it should be [display text](destination ur)
package.json
Outdated
"deploy:bcowfactory:mainnet": "forge script DeployBCoWFactory -vvvvv --rpc-url $MAINNET_RPC --broadcast --chain mainnet --private-key $MAINNET_DEPLOYER_PK --verify", | ||
"deploy:bcowfactory:testnet": "forge script DeployBCoWFactory -vvvvv --rpc-url $SEPOLIA_RPC --broadcast --chain sepolia --private-key $SEPOLIA_DEPLOYER_PK --verify", | ||
"deploy:bfactory:gnosis": "forge script DeployBFactory -vvvvv --rpc-url $GNOSIS --broadcast --chain gnosis --private-key $GNOSIS_DEPLOYER_PK --verify", |
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.
s/GNOSIS/GNOSIS_RPC/ ?
script/Registry.s.sol
Outdated
bCoWFactory = BCoWFactory(0xe8587525430fFC9193831e1113a672f3133C1B8A); | ||
bCoWHelper = BCoWHelper(0x0fd365F9Ed185512536E7dbfc7a8DaE43cD3CA09); | ||
bCoWFactory = BCoWFactory(0xf3916A8567DdC51a60208B35AC542F5226f46773); | ||
bCoWHelper = BCoWHelper(0x55DDf396886C85e443E0B5A8E42CAA3939E4Cf50); | ||
} else { | ||
// TODO: add Gnosis chain |
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.
TODO no longer applies
@@ -20,14 +17,16 @@ abstract contract Registry is Params { | |||
// TODO: redeploy | |||
if (chainId == 1) { | |||
// Ethereum Mainnet | |||
bFactory = BFactory(0xaD0447be7BDC80cf2e6DA20B13599E5dc859b667); |
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.
are we assuming BaseScript
won't use the regular factory?
package.json
Outdated
@@ -16,15 +16,18 @@ | |||
"build": "forge build", | |||
"build:optimized": "FOUNDRY_PROFILE=optimized forge build", | |||
"coverage": "forge coverage --match-path 'test/unit/**'", | |||
"deploy:bcowfactory:gnosis": "forge script DeployBCoWFactory -vvvvv --rpc-url $GNOSIS --broadcast --chain gnosis --private-key $GNOSIS_DEPLOYER_PK --verify", |
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.
👀 s/GNOSIS/GNOSIS_RPC/
deployed c250754 to mainnet, gnosis and sepolia.