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

Remove context from testthat Tests #148

Open
njlyon0 opened this issue May 1, 2024 · 1 comment
Open

Remove context from testthat Tests #148

njlyon0 opened this issue May 1, 2024 · 1 comment

Comments

@njlyon0
Copy link
Member

njlyon0 commented May 1, 2024

According to the chapter on testing in R Packages 2e, the context function is superseded in testthat version 3.0 and can be removed (context is provided by test file name and location in package). Not sure we'll get a flag from CRAN if we don't remove it but seems worthwhile to change while we're prepping for a new version release to CRAN.

@njlyon0 njlyon0 added this to the Submit to CRAN (`v 0.3.0`) milestone May 1, 2024
@brunj7 brunj7 removed this from the Submit to CRAN (`v 0.3.0`) milestone Jul 3, 2024
@brunj7
Copy link
Collaborator

brunj7 commented Jul 3, 2024

I removed it from the milestone has CRAN did not mention this. But in the long run it is probably an update we want to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants