scaffold-eth / scaffold-eth-2

Open source forkable Ethereum dev stack
https://scaffoldeth.io
MIT License
1.34k stars 837 forks source link

Migrate to foundry #867

Closed technophile-04 closed 3 months ago

technophile-04 commented 3 months ago

TODOs:

Testing steps:

Follow this branch README: https://github.com/scaffold-eth/scaffold-eth-2/tree/foundry-main?tab=readme-ov-file#-scaffold-eth-2

Next todos:

technophile-04 commented 3 months ago

812 maybe something we could handle in different PR nicely?

portdeveloper commented 3 months ago

Do you think if it could be any better if we could combine the following steps into one? image

Could yarn install && forge install --root packages/foundry and cp packages/foundry/.env.example packages/foundry/.env be one step in the install process for people who have just started learning onchain dev?

Maybe it could be one command: yarn setup or yarn buidl

carletex commented 3 months ago

This is great stuff @technophile-04 tysm!! I'll start testing and report back!

rin-st commented 3 months ago

Is it only for me that when I'm using metamask with foundry network, my setGreeting transactions are always pending?

Reset account fixed this

technophile-04 commented 3 months ago

Was talking with Carlos and just switched the base branch from main to foundry , thinking that https://github.com/scaffold-eth/scaffold-eth-2/pull/867#discussion_r1651262820 would be really nice if we get it working before we merge all the foundry changes to main branch and then announce the changes also was mentioned in https://github.com/scaffold-eth/scaffold-eth-2/discussions/868.

So for this PR we get this https://github.com/scaffold-eth/scaffold-eth-2/pull/867#discussion_r1651330259 solved merge it to foundry . And then tackle #812 merge it to foundry and after that finally merge foundry => main

technophile-04 commented 3 months ago

Just merged the postinstallchanges mentioned in https://github.com/scaffold-eth/scaffold-eth-2/pull/867#discussion_r1651330259 works nicely for me!. cc @Pabl0cks could you test it on windows once w/ without wsl and gitbash also if others could give another try too would be great 🙌

To test first run:

git clone https://github.com/scaffold-eth/scaffold-eth-2.git -b foundry-main scaffold-eth-foundry
cd scaffold-eth-foundry
yarn install && forge install --root packages/foundry

And then the three command yarn chain, yarn deploy, yarn start should work out of the box

rin-st commented 3 months ago

if others could give another try too would be great

for me it also works! 👍

damianmarti commented 3 months ago

Just merged the postinstallchanges mentioned in #867 (comment) works nicely for me!. cc @Pabl0cks could you test it on windows once w/ without wsl and gitbash also if others could give another try too would be great 🙌

To test first run:

git clone https://github.com/scaffold-eth/scaffold-eth-2.git -b foundry-main scaffold-eth-foundry
cd scaffold-eth-foundry
yarn install && yarn install && forge install --root packages/foundry

And then the three command yarn chain, yarn deploy, yarn start should work out of the box

Everything working great on Ubuntu 20.04!!

@technophile-04 Great job!!!

portdeveloper commented 3 months ago

Tried it on a win11 + git bash machine It works great, without any issues! 🚀🚀🚀 Incredible work @technophile-04 !!!

technophile-04 commented 3 months ago

Thanks all merging this in foundry branch 🙌