Skip to content

Conversation

amiecorso
Copy link
Contributor

@amiecorso amiecorso commented Aug 18, 2025

Adds an explicit profile for deployment builds. Employ optimizers for more efficient bytecode, pin settings that can affect build output for reproducibility.

via_ir = true // Route compilation through the Yul IR (intermediate rep) pipeine, generally get smaller bytecode, better optimization, lower gas at runtime
optimizer = true // use the compiler optimizer
optimizer_runs = 10000 // higher runs optimizes for better runtime gas at the expense of possibly more expensive one-time deployment costs, definitely a desirable choice to absorb onetime higher gas during deployment for lower per-call gas to users
evm_version = "cancun" // latest evm version compatible with solidity 0.8.26
bytecode_hash = "ipfs" // already the foundry default but doesn't hurt to pin this explicitly. has something to do with
offline = true // no fresh network fetches for dependencies - we want these fixed where we have them, and this expects that we already have all necessary dependencies installed at the correct commit locally when building

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Aug 18, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@amiecorso amiecorso merged commit fc5caaa into main Aug 18, 2025
7 checks passed
@amiecorso amiecorso deleted the amie/deployment-build-config branch August 18, 2025 21:07
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.

3 participants