Skip to content

Conversation

zerosnacks
Copy link
Member

@zerosnacks zerosnacks commented Aug 21, 2025

Motivation

Closes: #8340
Supersedes: #9930

Solution

Add Vyper template with deployCode, uses ICounter.sol interface which could perhaps be changed to an interface written in Vyper but not sure if this works well. Additionally the tests and scripts are written in Solidity.

Results in template here: https://github.com/zerosnacks/vyper-template-11383

With matching CI workflow here: https://github.com/zerosnacks/vyper-template-11383/actions/runs/17124914857/job/48574231647

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

Co-authored-by: startup-dreamer <prsumit35@gmail.com>
@zerosnacks zerosnacks marked this pull request as ready for review August 21, 2025 11:04
@zerosnacks zerosnacks changed the title feat: add --vyper to forge init to start with a Vyper project feat(forge): add --vyper to forge init to start with a Vyper project Aug 21, 2025
@zerosnacks zerosnacks added C-forge Command: forge Cmd-forge-init Command: forge init labels Aug 21, 2025
@zerosnacks zerosnacks moved this to Ready For Review in Foundry Aug 21, 2025
@zerosnacks zerosnacks requested a review from DaniPopes August 21, 2025 15:49
@zerosnacks zerosnacks enabled auto-merge (squash) August 25, 2025 11:05
@zerosnacks zerosnacks disabled auto-merge August 25, 2025 11:05
@zerosnacks
Copy link
Member Author

pending final reviews @yash-atreya / @grandizzy

Copy link
Collaborator

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

lgtm!

@zerosnacks zerosnacks merged commit ce40d54 into master Aug 25, 2025
22 checks passed
@zerosnacks zerosnacks deleted the zerosnacks/add-vyper-template branch August 25, 2025 13:04
@github-project-automation github-project-automation bot moved this from Ready For Review to Done in Foundry Aug 25, 2025
MerkleBoy pushed a commit to MerkleBoy/foundry that referenced this pull request Sep 17, 2025
…oject (foundry-rs#11383)

* add vyper template and test

Co-authored-by: startup-dreamer <prsumit35@gmail.com>

* make sure to include interface

* address feedback

* nits, python-version: "3.x" is required

* add basic smoke test for regular forge init to make sure no regression was introduced

* remove ids

---------

Co-authored-by: startup-dreamer <prsumit35@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge Cmd-forge-init Command: forge init
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

feat(forge init): add --vyper flag to initialize a basic Vyper project mirroring Counter
3 participants