Skip to content
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: admin fund amount #44

Merged
merged 4 commits into from
Nov 29, 2024
Merged

chore: admin fund amount #44

merged 4 commits into from
Nov 29, 2024

Conversation

lesterli
Copy link
Member

Summary

This PR updates to fund the admin address with 0.1ether and also:

  • replaced command readarray with a more portable way to create an array from the prefunded accounts that works on both macOS and Linux
  • .gitignore for .env.bak*, b/c running make l2-gen-addresses could generate the backup file env.bak-xxx if addresses already exist

Test Plan

launch local l1:

make l1-configure
make l1-launch
make l1-verify

update the .env:

L1_RPC_URL=http://localhost:18545
L1_BEACON_URL=http://localhost:15052
L1_CHAIN_ID=11155222
L1_FUNDED_PRIVATE_KEY=<the key's value in the configs/l1/l1-prefund-wallet.json>

test it and the output:

make l2-gen-addresses

Generated new addresses and updated .env file.
Funding ADMIN address: 0x5B46F34d400704EbcD408282c848d8c6e523EE88 with amount 0.1ether
...
Funding BATCHER address: 0xf2125cD1Bf90EC81EC0FEFc434d8C38AB4Fa5A39 with amount 1ether
...
Funding PROPOSER address: 0x22Cc0752894A7B7F9A8dAb1E94ac5d170aE84667 with amount 1ether


Funding complete.

use cast to verify, it should also return 0.1

cast balance <admin address in the .env> -e --rpc-url http://localhost:18545

@lesterli lesterli requested a review from a team as a code owner November 27, 2024 03:01
@lesterli lesterli merged commit a6fd108 into main Nov 29, 2024
1 check passed
@lesterli lesterli deleted the chore/admin-fund-amount branch November 29, 2024 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants