-
Notifications
You must be signed in to change notification settings - Fork 317
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
Allow Perl-style regex filters for tests #188
Conversation
I'd be tempted to use |
I'd be fine with that, but didn't want to steal |
I've updated to use |
Can you please rebase? |
Done. |
The diff doesn't look right to me - can you please double check? |
Oh, yea, that was weird. How does this look? |
@@ -1,5 +1,9 @@ | |||
# testthat 0.9.1.9000 | |||
|
|||
* `test_dir`, `test_package`, and `test_check` have an added `...` argument |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't look like you actually modified test_check()
(which is right) but can you remove from here?
Also house style is to follow function names with ()
, e.g.test_dir()
.
Meant to modify Added parentheses to function names in NEWS. |
Allow Perl-style regex filters for tests
Perl-style regular expressions are currently not allowed in the
filter
argument totest_dir
,test_package
, andtest_check
. This pull request adds aperl
argument (logical, defaultFALSE
) that enables Perl-style regular expressions, which is passed down togrepl
.