Add arg to skip with() in object_usage_linter()

[IndrajeetPatil]

Closes #1458

Example

library(lintr)

f <- tempfile()
writeLines(
  "test_fun <- function(df) {
      with(df, first_var + second_var)
     }",
  f
)

lint(f, object_usage_linter(skip_with = TRUE))

lint(f, object_usage_linter(skip_with = FALSE))
#> /private/var/folders/xr/v_vddzvs33q5wg7jv9mr__4h0000gn/T/Rtmp1OPl2d/file1557f756350e9:2:16: warning: [object_usage_linter] no visible binding for global variable 'first_var'
#>       with(df, first_var + second_var)
#>                ^~~~~~~~~
#> /private/var/folders/xr/v_vddzvs33q5wg7jv9mr__4h0000gn/T/Rtmp1OPl2d/file1557f756350e9:2:28: warning: [object_usage_linter] no visible binding for global variable 'second_var'
#>       with(df, first_var + second_var)
#>                            ^~~~~~~~~~

^{Created on 2022-09-22 with reprex v2.0.2}

P.S. A few .Rd changes are remnants of #1539, I think.

[AshesITR]

Isn't R CMD checks default skipWith = TRUE? Are there valid usages of with() that don't produce false positives?

[IndrajeetPatil]

@AshesITR You are right; I have changed the default for skip_with to TRUE, since setting is to FALSE is rarely going to not produce a false positive.

I also made some changes to the docs to close #941. The rest of the todos in that issue already seem to have been handled.