Consider modifying coverage output file

foundry-rs / starknet-foundry

Blazing fast toolkit for developing Starknet contracts.

https://foundry-rs.github.io/starknet-foundry/

MIT License

296 stars 143 forks source link

Consider modifying coverage output file #2407

Open ddoktorski opened 2 weeks ago

ddoktorski commented 2 weeks ago

Which components does the task require to be changed? (think hard pls)

snforge

Description

Currently the --coverage flag always creates and overwrites the coverage.lcov file. Consider either making the output file configurable or including a timestamp/hash of trace data files (or something similar) in the output file name. Refer to the initial discussion https://github.com/foundry-rs/starknet-foundry/pull/2369#discussion_r1721927076

MaksymilianDemitraszek commented 2 weeks ago

Imo timestamp is better tbh it is a standard for such files, hash seems like massive overcomplication.