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

Jump to bottom

Add output argument to deno coverage when used with --lcov #13236

Open
x80486 opened this issue Dec 30, 2021 · 2 comments
Open

Add output argument to deno coverage when used with --lcov #13236

x80486 opened this issue Dec 30, 2021 · 2 comments
Labels
feat good first issue l-rust

Comments

@x80486
Copy link

@x80486 x80486 commented Dec 30, 2021

I think it's better to provide an option to pipe the output to a file of deno converage cov_profile/ --lcov instead of using >, which is not supported in every operating system, AFAIK.

Something like deno converage cov_profile/ --lcov --output=path/to/lcov.info — possibly suppressing any output from lvoc, if any.
@bartlomieju
Copy link
Member

@bartlomieju bartlomieju commented Jan 4, 2022

I think this is reasonable request. We might skip --output and do what deno bundle does already:

// output to stdout
deno bundle main.js

// output to main.bundle.js
deno bundle main.js main.bundle.js

@bartlomieju bartlomieju changed the title Add --output option to deno coverage when used with --lcov Add output argument to deno coverage when used with --lcov Jan 4, 2022
@VishnuJin
Copy link
Contributor

@VishnuJin VishnuJin commented Jan 5, 2022

@bartlomieju I would like to work on this issue !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feat good first issue l-rust
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@bartlomieju @x80486 @VishnuJin