Skip to content

feat: npm publish anvil, cast and chisel #11945

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

feat: npm publish anvil, cast and chisel #11945

wants to merge 5 commits into from

Conversation

@grandizzy
Copy link
Collaborator

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@grandizzy grandizzy self-assigned this Oct 3, 2025
@o-az o-az force-pushed the o-az/npm-publish-rest branch from 3e39c31 to 143d674 Compare October 13, 2025 23:54
@jenpaff jenpaff moved this to In Progress in Foundry Oct 15, 2025
@guidanoli guidanoli mentioned this pull request Oct 17, 2025
Merged
@guidanoli
Copy link

This feature would be very useful to us at cartesi/rollups-contracts. We use Cannon to describe our smart contract deployments, and it runs forge and anvil under-the-hood. If we could have these binaries be distributed and versioned by Node.js project, this would be one less dependency requirement for our devs and users, and we would be able to scope Foundry versions locally! Do you also consider publishing a @foundry-rs/foundry package which includes all these binaries?

@grandizzy
Copy link
Collaborator Author

yes, we could include. Btw forge is already published in npm so you could give ot a try already, this PR tracks publish of the other Foundry artifacts

@o-az o-az force-pushed the o-az/npm-publish-rest branch 2 times, most recently from 343d714 to e45c88b Compare October 21, 2025 23:08
@o-az o-az force-pushed the o-az/npm-publish-rest branch from a47caf4 to a19c5ef Compare October 22, 2025 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DaniPopes DaniPopes Awaiting requested review from DaniPopes DaniPopes will be requested when the pull request is marked ready for review DaniPopes is a code owner

@mattsse mattsse Awaiting requested review from mattsse mattsse will be requested when the pull request is marked ready for review mattsse is a code owner

@zerosnacks zerosnacks Awaiting requested review from zerosnacks zerosnacks will be requested when the pull request is marked ready for review zerosnacks is a code owner

@onbjerg onbjerg Awaiting requested review from onbjerg onbjerg will be requested when the pull request is marked ready for review onbjerg is a code owner

@0xrusowsky 0xrusowsky Awaiting requested review from 0xrusowsky 0xrusowsky will be requested when the pull request is marked ready for review 0xrusowsky is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@grandizzy grandizzy

Labels

None yet

Projects

Foundry
Status: In Progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@grandizzy @guidanoli @o-az