Rename no_tab_linter to whitespace_linter (#1955)

* Rename no_tab_linter to whitespace_linter

* NEWS

* syntax error in test

* rename other references to no_tab_linter

* syntax in one more test

* trailing blank line

Author: MichaelChirico

DESCRIPTION

@@ -127,7 +127,6 @@ Collate:
     'namespace.R'
     'namespace_linter.R'
     'nested_ifelse_linter.R'
-    'no_tab_linter.R'
     'nonportable_path_linter.R'
     'numeric_leading_zero_linter.R'
     'object_length_linter.R'
@@ -169,6 +168,7 @@ Collate:
     'unused_import_linter.R'
     'use_lintr.R'
     'vector_logic_linter.R'
+    'whitespace_linter.R'
     'with.R'
     'with_id.R'
     'xml_nodes_to_lints.R'

NAMESPACE

@@ -136,6 +136,7 @@ export(unreachable_code_linter)
 export(unused_import_linter)
 export(use_lintr)
 export(vector_logic_linter)
+export(whitespace_linter)
 export(with_defaults)
 export(with_id)
 export(xml_nodes_to_lints)

NEWS.md

@@ -5,6 +5,7 @@
 * `single_quotes_linter()` is deprecated in favor of the more generalizable `quotes_linter()` (#1729, @MichaelChirico).
 * `unneeded_concatentation_linter()` is deprecated in favor of `unnecessary_concatenation_linter()` for naming consistency (#1707, @IndrajeetPatil).
 * `consecutive_stopifnot_linter()` is deprecated in favor of the more general (see below) `consecutive_assertion_linter()` (#1604, @MichaelChirico).
+* `no_tab_linter()` is deprecated in favor of `whitespace_linter()` for naming consistency and future generalization (#1954, @MichaelChirico).
 
 ## Bug fixes
 
@@ -65,6 +66,7 @@
    + `indentation_linter()`
    + `quotes_linter()`
    + `unnecessary_concatenation_linter()`
+   + `whitespace_linter()`
 
 ## New and improved features
 
@@ -152,6 +154,8 @@
 
 * `consecutive_assertion_linter()` (f.k.a. `consecutive_stopifnot_linter()`) now lints for consecutive calls to `assertthat::assert_that()` (as long as the `msg=` argument is not used; #1604, @MichaelChirico).
 
+* `whitespace_linter()` is simply `no_tab_linter()`, renamed. In the future, we plan to extend it to work for different whitespace preferences.
+
 ## Notes
 
 * {lintr} now depends on R version 3.5.0, in line with the tidyverse policy for R version compatibility.

R/lintr-deprecated.R

@@ -239,3 +239,16 @@ consecutive_stopifnot_linter <- function() {
   )
   consecutive_assertion_linter()
 }
+
+#' No tabs linter
+#' @rdname lintr-deprecated
+#' @export
+no_tab_linter <- function() {
+  lintr_deprecated(
+    old = "no_tab_linter",
+    new = "whitespace_linter",
+    version = "3.1.0",
+    type = "Linter"
+  )
+  whitespace_linter()
+}

R/no_tab_linter.R renamed to R/whitespace_linter.R

@@ -1,32 +1,35 @@
-#' No tab linter
+#' Whitespace linter
 #'
-#' Check that only spaces are used for indentation, not tabs. Much ink has been
-#' spilled on this topic, and we encourage you to check out references for more
-#' information.
+#' Check that the correct character is used for indentation.
+#'
+#' Currently, only supports linting in the presence of tabs.
+#'
+#' Much ink has been spilled on this topic, and we encourage you to check
+#'   out references for more information.
 #'
 #' @include make_linter_from_regex.R
 #'
 #' @examples
 #' # will produce lints
 #' lint(
 #'   text = "\tx",
-#'   linters = no_tab_linter()
+#'   linters = whitespace_linter()
 #' )
 #'
 #' # okay
 #' lint(
 #'   text = "  x",
-#'   linters = no_tab_linter()
+#'   linters = whitespace_linter()
 #' )
 #'
-#' @evalRd rd_tags("no_tab_linter")
+#' @evalRd rd_tags("whitespace_linter")
 #' @seealso [linters] for a complete list of linters available in lintr.
 #'
 #' @references
 #' - https://www.jwz.org/doc/tabs-vs-spaces.html
 #' - https://blog.codinghorror.com/death-to-the-space-infidels/
 #' @export
-no_tab_linter <- make_linter_from_regex(
+whitespace_linter <- make_linter_from_regex(
   regex = rex(start, zero_or_more(regex("\\s")), one_or_more("\t")),
   lint_type = "style",
   lint_msg = "Use spaces to indent, not tabs."

R/zzz.R

@@ -23,7 +23,6 @@ default_linters <- modify_defaults(
   indentation_linter(),
   infix_spaces_linter(),
   line_length_linter(),
-  no_tab_linter(),
   object_length_linter(),
   object_name_linter(),
   object_usage_linter(),
@@ -37,7 +36,8 @@ default_linters <- modify_defaults(
   T_and_F_symbol_linter(),
   trailing_blank_lines_linter(),
   trailing_whitespace_linter(),
-  vector_logic_linter()
+  vector_logic_linter(),
+  whitespace_linter()
 )
 
 #' Default undesirable functions and operators

inst/lintr/linters.csv

@@ -49,7 +49,7 @@ missing_argument_linter,correctness common_mistakes configurable
 missing_package_linter,robustness common_mistakes
 namespace_linter,correctness robustness configurable executing
 nested_ifelse_linter,efficiency readability
-no_tab_linter,style consistency default
+no_tab_linter,style consistency deprecated
 nonportable_path_linter,robustness best_practices configurable
 numeric_leading_zero_linter,style consistency readability
 object_length_linter,style readability default configurable executing
@@ -93,4 +93,5 @@ unneeded_concatenation_linter,style readability efficiency configurable deprecat
 unreachable_code_linter,best_practices readability
 unused_import_linter,best_practices common_mistakes configurable executing
 vector_logic_linter,default efficiency best_practices
+whitespace_linter,style consistency default
 yoda_test_linter,package_development best_practices readability