Skip to content

Conversation

grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Sep 29, 2025

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@DaniPopes
Copy link
Member

in forge flatten we have a fallback for using the old flattener:

let flattened = match flattener {

can we unify all the invocations to a single common util function that handles this?

@grandizzy
Copy link
Collaborator Author

in forge flatten we have a fallback for using the old flattener:

let flattened = match flattener {

can we unify all the invocations to a single common util function that handles this?

good point, added in d60e7b2 and extended to cast src as well

@grandizzy grandizzy marked this pull request as ready for review September 30, 2025 05:15
@grandizzy grandizzy merged commit 6d69231 into foundry-rs:master Sep 30, 2025
16 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Sep 30, 2025
@grandizzy grandizzy deleted the issue-11754 branch September 30, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

forge verify-contract --flatten produces different flattened code than forge flatten command
3 participants