-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: accept ETH_RPC_URL env as fork-url alias #8972
base: master
Are you sure you want to change the base?
Conversation
The integration tests are a rabbit hole to fix. This PR isn't as simple as I was expecting and currently I don't have time to fix everything, so I'll return to it in a few days. |
related: #7755 |
@CodeSandwich thanks for your contribution. Do you still have the bandwidth to work on this? |
@yash-atreya Thanks for the reminder, TBH I forgot about this PR. I think that I may be able to take another stab this weekend unless somebody wants to take it over. |
@yash-atreya I'm giving up on this PR, feel free to pass it to somebody else or close it for good. I'm simply unable to work with the integration tests. Running them on my local machine with ( I wish all the best to Foundry, I like this tool very much and I want it to succeed. I think that the developer experience may be harming the project, for example I as of now can't see myself contributing to its codebase and probably other outside contributors are facing a similar barrier. I hope that at least the core contributors manage to achieve a good experience and aren't slowed down by issues around the test suite. |
Thank you for the contribution and much-needed feedback on local testing for contributors. Will keep this in mind going forward. |
Motivation
Currently commands like
forge script
don't read theETH_RPC_URL
env variable, you need to toforge script --rpc-url "$ETH_RPC_URL"
(or--fork-url
or-f
,--rpc-url
is just one of the aliases). This is unlike all other commands likeforge create
, which acceptETH_RPC_URL
as the--rpc-url
alternative.Solution
Add support for
ETH_RPC_URL
wherever--fork-url
is accepted.