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 test_coverage_file to run covr::report on the currently active file #1758

Merged
merged 1 commit into from
Apr 9, 2018

Conversation

jimhester
Copy link
Member

Fixes #1755

This works pretty well, the only downside is you need to click through the first table even if there is only one file which is slightly cumbersome. The upside is if you pass more than one file it works nicely.

Fixing this would require some changes in covr.

@jimhester jimhester requested a review from hadley April 6, 2018 19:18
Copy link
Member

@hadley hadley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe just capture the single file use case in a covr issue so we can revisit at some point?

@jimhester
Copy link
Member Author

Sounds good, did so r-lib/covr#308

@jimhester jimhester merged commit ab568e9 into master Apr 9, 2018
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.

2 participants