Warning exists when reprex(... style = FALSE)

[beauchamplab]

With style=FALSE, users would like to render without styler. I wonder if "Install the styler package" should still show up?

reprex::reprex({ print("") }, style = FALSE)
#> ✖ Install the styler package in order to use `style = TRUE`.
#> ℹ Non-interactive session, setting `html_preview = FALSE`.
#> ℹ Rendering reprex...

^{Created on 2024-02-02 with reprex v2.1.0}

[jennybc]

Yeah this appears to be a thinko in this helper. It doesn't seem to consult whether style is TRUE or FALSE 🙈

reprex/R/reprex_impl.R

Lines 129 to 137 in e1f65e9

	style_requires_styler <- function(style) {
	if (!requireNamespace("styler", quietly = TRUE)) {
	reprex_danger("
	Install the {.pkg styler} package in order to use
	{.code style = TRUE}.")
	style <- FALSE
	}
	invisible(style)
	}

[beauchamplab]

Our users asked us if they should install styler since they got this "warning". Maybe:

style_requires_styler <- function(style) { 
   if (style && !requireNamespace("styler", quietly = TRUE)) { 
     reprex_danger(" 
       Install the {.pkg styler} package in order to use 
       {.code style = TRUE}.") 
     style <- FALSE 
   } 
   invisible(style) 
 } 

?

[jennybc]

Yes LGTM. Would you like to make a PR?