Skip to content

feat: add deployment-foundry-profile input#24

Open
hudsonhrh wants to merge 1 commit intomainfrom
hudsonhrh/deployment-foundry-profile
Open

feat: add deployment-foundry-profile input#24
hudsonhrh wants to merge 1 commit intomainfrom
hudsonhrh/deployment-foundry-profile

Conversation

@hudsonhrh
Copy link
Copy Markdown
Contributor

Summary

  • Adds a deployment-foundry-profile input to the reusable workflow
  • Sets FOUNDRY_PROFILE env var in both testnet and mainnet deploy steps
  • Backward-compatible: defaults to empty string (uses default profile)

This allows callers to use an optimized Foundry profile for deployments (e.g., with the Solidity optimizer enabled) while keeping the default profile for tests. Useful when the optimizer must be disabled for tests (e.g., due to vm.roll() bugs with via_ir) but enabled for deployments to stay within contract size limits.

Usage

jobs:
  foundry-cicd:
    uses: BreadchainCoop/etherform/.github/workflows/_foundry-cicd.yml@main
    with:
      deploy-on-main: true
      deployment-foundry-profile: 'production'

Generated with Claude Code

@rubydusa rubydusa force-pushed the hudsonhrh/deployment-foundry-profile branch from e4ef95f to 7ca0872 Compare April 2, 2026 08:57
Allow callers to specify a Foundry profile for deployment builds by
setting the FOUNDRY_PROFILE environment variable. This enables using
optimized profiles (e.g., with solc optimizer enabled) for deployments
while keeping the default profile for tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rubydusa rubydusa force-pushed the hudsonhrh/deployment-foundry-profile branch from 7ca0872 to 66eac99 Compare April 2, 2026 09:02
@rubydusa
Copy link
Copy Markdown
Contributor

rubydusa commented Apr 2, 2026

6 line diff turned into 5 because mainnet deployment was removed in #29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants