Skip to content

Conversation

@gustavoguimaraes
Copy link
Contributor

What does this PR do?

remove obsolete files from reth-exex folder plus organize contracts folder

* get rid of reth-exex folder
* organize contracts folder
* get rid of reth-exexs
* organize contracts folder
Copy link
Collaborator

@jorgemmsilva jorgemmsilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all the "non-latest" run-*.json files could probably be deleted 😛

@@ -1,6 +1,6 @@
[submodule "contracts/lib/forge-std"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this file can now be deleted? since we have the .gitmodules file at project root

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is Foundry specific submodules. It needs these for tests to work.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also foundry keep the run-* files as historical deployment artifacts. I will keep these in case we need to check which contracts were deployed to which chains while we are testing.

Copy link
Contributor

@WillPapper WillPapper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @gustavoguimaraes!

@@ -0,0 +1 @@
ETH_PRIVATE_KEY= No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we fill this with the devnet default? https://docs.optimism.io/chain/testing/dev-node#additional-info. This also works for local Anvil nodes

That way, auto-deployment to the devnet will work easily. Likewise, we should default the RPC URL to the devnet (http://localhost:9545)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. Will do.

@gustavoguimaraes gustavoguimaraes merged commit a929e6b into main Oct 16, 2024
@gustavoguimaraes gustavoguimaraes deleted the gus/cleanup branch October 16, 2024 17:26
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.

4 participants