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

add fuzz show_log docs #1238

Merged
merged 1 commit into from
Aug 6, 2024
Merged

add fuzz show_log docs #1238

merged 1 commit into from
Aug 6, 2024

Conversation

Azleal
Copy link
Contributor

@Azleal Azleal commented Jul 9, 2024

updated docs related to foundry-rs/foundry#8387

@Azleal Azleal requested review from Evalir, mattsse and onbjerg as code owners July 9, 2024 13:00
@Azleal
Copy link
Contributor Author

Azleal commented Jul 9, 2024

hi @grandizzy @mattsse , I'm not sure where this Environment come from. So I left it blank for show_logs. please tell me if Environment should be here.

##### `include_push_bytes`
- Type: boolean
- Default: true
- Environment: `FOUNDRY_FUZZ_INCLUDE_PUSH_BYTES`

@grandizzy
Copy link
Collaborator

hi @grandizzy @mattsse , I'm not sure where this Environment come from. So I left it blank for show_logs. please tell me if Environment should be here.

##### `include_push_bytes`
- Type: boolean
- Default: true
- Environment: `FOUNDRY_FUZZ_INCLUDE_PUSH_BYTES`

Configs within profile can be passed as env vars, for example you can test that new one works if FOUNDRY_FUZZ_SHOW_LOGS=true

show-logs doc

environment
@Azleal
Copy link
Contributor Author

Azleal commented Jul 11, 2024

Configs within profile can be passed as env vars, for example you can test that new one works if FOUNDRY_FUZZ_SHOW_LOGS=true

yeah. i tried FOUNDRY_FUZZ_SHOW_LOGS=true forge test --vv, it worked as expected. and I updated the docs.

@Azleal
Copy link
Contributor Author

Azleal commented Aug 6, 2024

hi @mattsse , could you please review this PR to see if it could be merged.

@Evalir Evalir merged commit adcec64 into foundry-rs:master Aug 6, 2024
1 check passed
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