Hadley Wickham
Hadley Wickham
I realised that I don't actually like the function signature of `rlang::local_binding()` because you almost always call it with an environment: ```R # e.g. instead of this: local_bindings(is_testing = FALSE,...
Oh yeah that's true. Still I think we should also provide in withr; probably just via copy and paste.
IMO we don't need to return `old` from `local_bindings()`, since I don't think people actually use the return values. And I looked at a couple of withr functions and it...
Going off my vague memory, it's probably because quarto vignettes are built in a temporary directory. I think you'll need to add the child to external resources?
Oops, yup.
Implementing this should be straightforward following the pattern of `imap_lgl()`, `imap_int()` etc.
Should probably mimic https://httr2.r-lib.org/reference/req_retry.html and provide both `max_tries` and `max_seconds`. Would be useful to have this also as something you can use on a block of code, not just a...
There is already a `max_tries` argument, but it's hard to see because it's an argument to `rate_backoff()`: ``` r library(purrr) f ▆ #> 1. └─insistently(f)(1) #> 2. └─purrr::rate_sleep(rate, quiet =...
That release was performed by BDR, not by us, but I find it hard to believe that it's responsible for your test failures.
It would be useful for me if you could inline the screenshot as well as including a link to the rendered site and the `_pkgdown.yml`.