Skip to content

Commit 0daaede

Browse files
clean keyword and brace related lints
1 parent 4f5e30a commit 0daaede

File tree

2 files changed

+11
-13
lines changed

2 files changed

+11
-13
lines changed

.lintr

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ linters: linters_with_tags(
88
line_length_linter(120L),
99
namespace_linter = NULL,
1010
nested_ifelse_linter = NULL,
11+
# TODO: remove this once the lint message is fixed
12+
nzchar_linter = NULL,
1113
object_name_linter = NULL,
1214
object_length_linter(70L),
1315
object_overwrite_linter = NULL,

R/io.R

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -32,21 +32,17 @@ transform_utf8_one <- function(path, fun, dry) {
3232
identical <- identical_content && !file_with_info$missing_EOF_line_break
3333
if (!identical) {
3434
switch(dry,
35-
"fail" = {
36-
rlang::abort(
37-
paste0(
38-
"File `", path, "` would be modified by styler and argument dry",
39-
" is set to 'fail'."
40-
),
41-
class = "dryError"
42-
)
43-
},
44-
"on" = {
35+
fail = rlang::abort(
36+
paste0(
37+
"File `", path, "` would be modified by styler and argument dry",
38+
" is set to 'fail'."
39+
),
40+
class = "dryError"
41+
),
42+
on = {
4543
# don't do anything
4644
},
47-
"off" = {
48-
write_utf8(new, path)
49-
},
45+
off = write_utf8(new, path),
5046
{
5147
# not implemented
5248
}

0 commit comments

Comments
 (0)