New linter for tidyverse return style

[MichaelChirico]

Here's another thing straight out of the style guide that we don't lint for, that styler fixes:

styler::style_text("
foo <- function(x) {
  if (x > 1) return(x)
  return(x - 1)
}
")
# foo <- function(x) {
#   if (x > 1) {
#     return(x)
#   }
#   return(x - 1)
# }

https://style.tidyverse.org/syntax.html#inline-statements

Function calls that affect control flow (like return(), stop() or continue) should always go in their own {} block:

[MichaelChirico]

I guess this should be a new parameter in return_linter()?

[AshesITR]

This seems to better suit brace_linter() since the advice is regarding placement of braces.