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

[Feature Request] aptos move coverage needs to be provide useful output when piped to a file #14377

Closed
brmataptos opened this issue Aug 22, 2024 · 0 comments · Fixed by #14449
Assignees
Labels
enhancement New feature or request stale-exempt Prevents issues from being automatically marked and closed as stale

Comments

@brmataptos
Copy link
Contributor

🚀 Feature Request

To reproduce:

cd aptos-move/move-examples/veiled_coin
aptos move test --coverage --named-addresses 'veiled_coin=0x424'
aptos move coverage source --named-addresses 'veiled_coin=0x424' --module veiled_coin

Ok, the output has color showing lines that presumably show lines covered (green) or not (red).

But pipe that to a file, or even the less command, and the color goes away, leaving this command pretty useless

aptos move coverage source --named-addresses 'veiled_coin=0x424' --module veiled_coin |& less -P
aptos move coverage source --named-addresses 'veiled_coin=0x424' --module veiled_coin >& test.out

We also should have a non-color output to handle people with visual issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale-exempt Prevents issues from being automatically marked and closed as stale
Projects
None yet
2 participants