-
Notifications
You must be signed in to change notification settings - Fork 759
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add DT to imports #2085
Comments
Except you don't need it at all when using covr on travis and DT is a fairly heavy package. @jennybc was actually the person who prompted me to move it to Suggests, back when reporting relied on shiny (r-lib/covr#53 (comment)) There have been a number of issues related to this, r-lib/covr#269, r-lib/covr#343 etc. I may improve the error message when DT is not installed, but I think having it in Suggests is still correct. |
Hmmm, good points. Maybe we should add to devtools’ imports? |
Yeah, we could probably do that. |
This problem struck again in the class I'm teaching today. |
This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/ |
Since you almost always need it
The text was updated successfully, but these errors were encountered: