Skip to content

fix(forge): preprocess mocks declared in test file #12516

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 1 commit into
base: master
Choose a base branch
from
Draft

fix(forge): preprocess mocks declared in test file #12516

wants to merge 1 commit into from
+130 −41

Conversation

@grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Nov 10, 2025 •
edited
Loading

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@grandizzy grandizzy mentioned this pull request Nov 10, 2025
Merged
grandizzy added a commit to foundry-rs/compilers that referenced this pull request Nov 10, 2025
@grandizzy
fix: always mark mocks as dirty (#335)
669d6c3 
ref foundry-rs/foundry#12452
- mark mock file edges + own mock file as dirty
- required for foundry-rs/foundry#12516
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

No one assigned

Labels

None yet

Projects

Foundry
Status: No status

Milestone

No milestone

2 participants

@grandizzy