foundry icon indicating copy to clipboard operation
foundry copied to clipboard

feat: add support for basic vyper project using forge init --vyper

Open startup-dreamer opened this issue 10 months ago • 7 comments

Closes: #8340

This PR aims to add forge init --vyper command which instantiate a basic vyper project like this project as suggested here https://github.com/foundry-rs/foundry/issues/8340#issuecomment-2289276286.

~~This command mimics forge init --template https://github.com/Patronum-Labs/foundry-vyper currently the standard way of initializing a vyper project.~~

PR Checklist

  • [x] Added Tests
  • [x] Added Documentation
  • [ ] Breaking changes

startup-dreamer avatar Feb 21 '25 05:02 startup-dreamer

Makes sense, I'll do the changes.

startup-dreamer avatar Feb 21 '25 10:02 startup-dreamer

Hey @startup-dreamer, if time permits - would be great to get this in! Thanks

zerosnacks avatar Mar 27 '25 16:03 zerosnacks

Hey @startup-dreamer, if time permits - would be great to get this in! Thanks

I will try to ship this by the end of weekend.

startup-dreamer avatar Mar 27 '25 22:03 startup-dreamer

Hi @zerosnacks, I did some preliminary work and am trying to test these changes by building binaries and running {local_path_to_foundry}/target/forge init --vyper. However, the Forge binary is not updating according to my latest code changes. I’ve tried everything, including rebuilding libs again from scratch and running cargo clean, but it’s stuck with partial updates. Do you have any idea how I can fix this?

startup-dreamer avatar Mar 29 '25 20:03 startup-dreamer

Thanks! This is looking great

zerosnacks avatar Mar 31 '25 08:03 zerosnacks

Hey @zerosnacks! I can continue working on this — I'd also love for this feature to go out ASAP. Just let me know if my approach and semantics look good to you; if not, I’m happy to make changes.

startup-dreamer avatar Apr 18 '25 19:04 startup-dreamer

Hey @startup-dreamer!

My apologies for my late reply! This got lost in all my notifications

Directionally this is looking good but it includes a few functions / files that are not required anymore

zerosnacks avatar Jun 05 '25 11:06 zerosnacks

Hi @startup-dreamer, would you be up for making the proposed modifications or would you like me to take this over? Thanks!

zerosnacks avatar Jul 02 '25 10:07 zerosnacks

Closing in favor of https://github.com/foundry-rs/foundry/pull/11383

Added you as a co-author

Thanks!

zerosnacks avatar Aug 21 '25 11:08 zerosnacks