chore: add L1 services to docker-compose

horsefacts commented 1 year ago

Motivation

Our existing docker-compose file only deploys our L2 contracts on an OP mainnet fork. It should also create an Ethereum mainnet fork and deploy the L1 FnameResolver.

Change Summary

Add L1 anvil and deployer services to docker-compose.yml.

Merge Checklist

Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The PR title adheres to the conventional commits standard
[x] The PR has been tagged with change type label(s) (i.e. documentation, feature, bugfix, or chore)
[x] The PR's changes adhere to all the requirements in the contribution guidelines
[x] All commits have been signed

PR-Codex overview

Detailed summary

This PR focuses on updating the docker-compose.yml file. The notable changes include:

Adding new services l2-anvil, l2-deployer, l1-anvil, and l1-deployer
Updating the command, environment variables, volumes, and ports for the new services
Removing the old services anvil and deployer
Updating the entrypoint for the l2-deployer and l1-deployer services
Adding new volumes for the l1-anvil and l2-anvil services

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

github-actions[bot] commented 1 year ago

Coverage after merging horsefacts/l1-deploy-docker into main will be

98.65%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src
Bundler.sol	100%	100%	100%	100%
FnameResolver.sol	100%	100%	100%	100%
IdRegistry.sol	100%	100%	100%	100%
KeyRegistry.sol	100%	100%	100%	100%
StorageRegistry.sol	100%	100%	100%	100%
src/lib
EIP712.sol	50%	100%	50%	50%	19
Signatures.sol	100%	100%	100%	100%
TransferHelper.sol	0%	0%	0%	0%	12, 17, 20, 20, 20
TrustedCaller.sol	100%	100%	100%	100%
src/validators
SignedKeyRequestValidator.sol	100%	100%	100%	100%

horsefacts commented 1 year ago

Should we update the .env.example file while we're here?

@sds Do you see anything specific that's missing? I added the L1_ and L2_ RPC variables in an earlier PR, so I think everything we use is defined already.

sds commented 1 year ago

Oh sorry, I was looking at this PR in isolation. Nevermind!

farcasterxyz / contracts