`document(quiet = TRUE)` is too quiet when a required package is not installed

[haugedal]

When running document() with the argument quiet = TRUE on a package where another required package is not installed you get the following “question”:

Selection:

With no information about what you are answering/selecting or how to answer/select.

The unsuppressed question with selections is as follows:

ℹ The package "packageName" is required.
✖ Would you like to install it?

1: Yes
2: No

It seems reasonable to show this information when running document(), even with quiet = TRUE.

[jennybc]

I agree it is reasonable to surface this, but this is also relatively low priority. I think an attempt to do any other package development task will likely also surface this problem in a less cryptic way.

I see this in roxygen2::roxygenize():

https://github.com/r-lib/roxygen2/blob/066c3a504ef94e9fcdf3a133706e59e184dfaa27/R/roxygenize.R#L53-L55

if (!is_interactive()) {
    withr::local_options(warn = 1)
}

So the easiest fix would be to call roxygen2::roxygenize() as if non-interactive when quiet = TRUE to get whatever the intended experience is in a non-interactive session.

[hadley]

I think the root problem is that roxygenise() doesn't offer it's own quiet argument, so usethis uses this hack:

  if (quiet) {
    output <- file_temp()
    withr::defer(file_delete(output))
    withr::local_output_sink(output)
  }

That is an overly aggressive way of quieting.

I think the question is then really if devtools should even attempt to offer a quiet argument here.

But that's a big thorny question and unfortunately I don't think it's ever going to be worth our time to think that through, so I'm going to close this issue