-
Notifications
You must be signed in to change notification settings - Fork 25
Small cleanup: with(future::plan(...), local = TRUE) #240
Copy link
Copy link
Open
Description
A minor comment, but these days, you can replace:
httpgd/tests/testthat/helper-server.R
Lines 7 to 14 in 087ebf5
| (function() { | |
| future::plan("multisession") | |
| v <- future::value(future::future({ | |
| httr::GET(...) | |
| })) | |
| future::plan("sequential") | |
| v | |
| })() |
with:
(function() {
with(future::plan("multisession"), local = TRUE)
future::value(future::future({
httr::GET(...)
}))
})() Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels