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

forge build --skip-tests #3366

Closed
tynes opened this issue Sep 26, 2022 · 1 comment · Fixed by #3370
Closed

forge build --skip-tests #3366

tynes opened this issue Sep 26, 2022 · 1 comment · Fixed by #3370
Assignees
Labels
C-forge Command: forge Cmd-forge-build Command: forge build T-feature Type: feature

Comments

@tynes
Copy link
Contributor

tynes commented Sep 26, 2022

Component

Forge

Describe the feature you would like

It would be nice to have a flag on forge build that skips building the tests. As our test suite grows, it results in the build being more slow. This would be useful for when the artifacts are desired, we use them as part of a build process to build bindings in go. We don't need to build the tests for this case.

There are a few ways to do this - skipping files that have .t.sol in the filename, or looking for the IS_TEST function in the contract ABI.

Potentially could apply the same thing for scripts

Additional context

No response

@tynes tynes added the T-feature Type: feature label Sep 26, 2022
@gakonst gakonst added this to Foundry Sep 26, 2022
@gakonst gakonst moved this to Todo in Foundry Sep 26, 2022
@mattsse
Copy link
Member

mattsse commented Sep 26, 2022

supportive of that.

this can be added very easily

@mattsse mattsse self-assigned this Sep 26, 2022
@rkrasiuk rkrasiuk added C-forge Command: forge Cmd-forge-build Command: forge build labels Sep 26, 2022
Repository owner moved this from Todo to Done in Foundry Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge Cmd-forge-build Command: forge build T-feature Type: feature
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants