Skip to content

fix: foundry version#580

Merged
mrekucci merged 1 commit intomainfrom
ff
Jan 28, 2025
Merged

fix: foundry version#580
mrekucci merged 1 commit intomainfrom
ff

Conversation

@mrekucci
Copy link
Copy Markdown
Contributor

@mrekucci mrekucci commented Jan 23, 2025

Describe your changes

Uses the same Foundry and NodeJS version in the CI as in the contracts deployer job.

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

@mrekucci mrekucci marked this pull request as ready for review January 23, 2025 20:24
@mrekucci mrekucci force-pushed the ff branch 6 times, most recently from f0ae5b6 to d658e15 Compare January 23, 2025 21:37
@mrekucci mrekucci closed this Jan 23, 2025
@mrekucci mrekucci reopened this Jan 23, 2025
@mrekucci mrekucci force-pushed the ff branch 10 times, most recently from 8a56653 to 8ae2be0 Compare January 24, 2025 17:30
@ckartik
Copy link
Copy Markdown
Contributor

ckartik commented Jan 27, 2025

Closing as this is fixed by: #584

@ckartik ckartik closed this Jan 27, 2025
@mrekucci mrekucci reopened this Jan 27, 2025
@mrekucci
Copy link
Copy Markdown
Contributor Author

Closing as this is fixed by: #584

This PR is not directly related to the CI problem, but ensures that the same versions are used across the board.

@mrekucci mrekucci merged commit c23fefc into main Jan 28, 2025
@mrekucci mrekucci deleted the ff branch January 28, 2025 17:39
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