-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update default .env
files
#205
Conversation
.env
files
Signed-off-by: ricott1 <[email protected]>
Signed-off-by: ricott1 <[email protected]>
Signed-off-by: ricott1 <[email protected]>
7c080b2
to
d4ca137
Compare
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.
Approving PR!
Just one small suggestion.
Inside .env.mainnet
we currently have two lines:
L2_RPC_URL=https://lisk.drpc.org
#L2_RPC_URL=https://rpc.api.lisk.com
Probably we should only have one line using lisk.com
url:
L2_RPC_URL=https://rpc.api.lisk.com
@matjazv @AndreasKendziorra Regarding the RPC URLs, should we consistently use the |
I vote for yes. Regarding L1 endpoints, we're currently using |
No strong opinion on this one. Either way is fine. |
* Update L1_FORK_RPC_URL and L2_FORK_RPC_URL default values Signed-off-by: ricott1 <[email protected]> * Add .env.devnet file and update README Signed-off-by: ricott1 <[email protected]> * Update devnet env file and README; Improve comments in env files Signed-off-by: ricott1 <[email protected]> * Make default URLs consistent * Remove unused fork rpc URLs from public chain .env files * Remove value of unused TEST_NETWORK_MNEMONIC variable * Update the deployment description in the README --------- Signed-off-by: ricott1 <[email protected]>
#203) * feat: add WBTC deployment script and addresses for mainnet and sepolia Signed-off-by: ricott1 <[email protected]> * Update WBTC address in testnet env file Signed-off-by: ricott1 <[email protected]> * Fix log typo Signed-off-by: ricott1 <[email protected]> * Update default `.env` files (#205) * Update L1_FORK_RPC_URL and L2_FORK_RPC_URL default values Signed-off-by: ricott1 <[email protected]> * Add .env.devnet file and update README Signed-off-by: ricott1 <[email protected]> * Update devnet env file and README; Improve comments in env files Signed-off-by: ricott1 <[email protected]> * Make default URLs consistent * Remove unused fork rpc URLs from public chain .env files * Remove value of unused TEST_NETWORK_MNEMONIC variable * Update the deployment description in the README --------- Signed-off-by: ricott1 <[email protected]> * feat: add WBTC deployment script and addresses for mainnet and sepolia Signed-off-by: ricott1 <[email protected]> * Update 9_deployWBTCContract.sh --------- Signed-off-by: ricott1 <[email protected]>
What was the problem?
The default .env files contained some values requiring an Infura API key and in general were not correctly pointing to the relevant chain.
How was it solved?
We updated the .env files accordingly.