Skip to content

Align Solidity source contracts with the solidity style guide#66

Open
holyfuchs wants to merge 2 commits into
mainfrom
holyfuchs/solidity-style-guide
Open

Align Solidity source contracts with the solidity style guide#66
holyfuchs wants to merge 2 commits into
mainfrom
holyfuchs/solidity-style-guide

Conversation

@holyfuchs

Copy link
Copy Markdown
Member

Align Solidity source contracts with the project style guide

Summary

Brings the contracts into conformance with the standard Solidity style guide.
https://docs.soliditylang.org/en/latest/style-guide.html

# FOUNDRY_BYTECODE_HASH=none strips the metadata hash, which otherwise
# always differs because it hashes the source text.
export FOUNDRY_BYTECODE_HASH=none

# 1) Capture bytecode on main.
git checkout main
forge build
forge inspect src/FCMVault.sol:FCMVault deployedBytecode > /tmp/before_vault.txt
forge inspect src/YieldTokenOracle.sol:YieldTokenOracle deployedBytecode > /tmp/before_oracle.txt

# 2) Capture bytecode on this branch.
git checkout holyfuchs/solidity-style-guide
forge build
forge inspect src/FCMVault.sol:FCMVault deployedBytecode > /tmp/after_vault.txt
forge inspect src/YieldTokenOracle.sol:YieldTokenOracle deployedBytecode > /tmp/after_oracle.txt

# 3) Compare — no output means the bytecode is identical.
diff /tmp/before_vault.txt /tmp/after_vault.txt
diff /tmp/before_oracle.txt /tmp/after_oracle.txt

@holyfuchs holyfuchs force-pushed the holyfuchs/solidity-style-guide branch from 9d8680c to f89f68c Compare June 24, 2026 20:33
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.

1 participant