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 an invert argument to find_tests_scripts #239

Merged
merged 2 commits into from Sep 30, 2015

Conversation

Projects
None yet
3 participants
@jimhester
Member

jimhester commented Mar 24, 2015

This allows selecting only tests which do not match a regular
expression.

Add an invert argument to find_tests_scripts
This allows selecting only tests which do _not_ match a regular
expression.

@jimhester jimhester referenced this pull request Mar 24, 2015

Merged

Test dots #747

@coveralls

This comment has been minimized.

coveralls commented Mar 24, 2015

Coverage Status

Changes Unknown when pulling 0b623ec on jimhester:invert_filter into * on hadley:master*.

@@ -62,17 +62,24 @@ source_test_helpers <- function(path, env = globalenv()) {
#' Find the test files.
#' @param path path to tests
#' @param filter cf \code{\link{test_dir}}
#' @param invert If \sQuote{TRUE} return files which do _not_ match.

This comment has been minimized.

@hadley

hadley Sep 24, 2015

Member

\emph{}?

@hadley

This comment has been minimized.

Member

hadley commented Sep 24, 2015

Can you add a bullet to NEWS please?

@hadley

This comment has been minimized.

Member

hadley commented Sep 29, 2015

If you can get this in soon, it's likely to make it into the next version

@jimhester

This comment has been minimized.

Member

jimhester commented Sep 29, 2015

Done c1fcdd0, thanks for the reminder!

hadley added a commit that referenced this pull request Sep 30, 2015

Merge pull request #239 from jimhester/invert_filter
Add an invert argument to find_tests_scripts

@hadley hadley merged commit 3b71998 into r-lib:master Sep 30, 2015

2 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment