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.

Sign up for GitHub

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

use requireNamespace for optional test #81

Merged
merged 2 commits into from
May 11, 2022
Merged

use requireNamespace for optional test #81

merged 2 commits into from
May 11, 2022

Conversation

MichaelChirico
Copy link
Contributor

@MichaelChirico MichaelChirico commented Mar 4, 2022 •
edited
Loading

require() throws a warning if not installed, this will be an error for options(warn = 2).

Also add the package to Suggests to indicate its usage in the tests.

@MichaelChirico
use requireNamespace for optional test
f12d0a1 
require() throws a warning if not installed, this will be an error for options(warn = 2)
@MichaelChirico
add to Suggests
96677b3
@jeremyrcoyle jeremyrcoyle merged commit ed72f8a into tlverse:master May 11, 2022
@jeremyrcoyle
Copy link
Collaborator

thanks!

@MichaelChirico MichaelChirico deleted the patch-1 branch May 11, 2022 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MichaelChirico @jeremyrcoyle