Skip to content
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.

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

Decouple Foundry build and project format from JS Solidity test runner #487

Closed
agostbiro opened this issue Jun 3, 2024 · 0 comments · Fixed by #572
Closed

Decouple Foundry build and project format from JS Solidity test runner #487

agostbiro opened this issue Jun 3, 2024 · 0 comments · Fixed by #572
Assignees

Comments

@agostbiro
Copy link
Member

agostbiro commented Jun 3, 2024

Problem

The JS Solidity test runner executes bytecode compiled with Hardhat, but it still depends on the Foundry project format (file layout, config) and build system to provide the following functionality (possibly incomplete):

Definition of Done

The Foundry build and project format is decoupled from JS Solidity test runner. TBD what functionality to preserve

@github-project-automation github-project-automation bot moved this to Unconfirmed in EDR Jun 3, 2024
@agostbiro agostbiro self-assigned this Jun 3, 2024
@agostbiro agostbiro moved this from Unconfirmed to Todo in EDR Jun 3, 2024
@agostbiro agostbiro moved this from Todo to In Progress in EDR Jun 3, 2024
@agostbiro agostbiro moved this from In Progress to Todo in EDR Jun 4, 2024
@fvictorio fvictorio added this to the Solidity tests, phase 2 milestone Jul 26, 2024
@agostbiro agostbiro linked a pull request Jul 31, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from Todo to Done in EDR Aug 1, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants