Skip to content

Commit ba99fb2

Browse files
committed
minor linting
1 parent 806d8cb commit ba99fb2

File tree

3 files changed

+15
-6
lines changed

3 files changed

+15
-6
lines changed

R/aaa_models.R

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,9 @@ check_model_doesnt_exist <- function(model) {
127127
}
128128

129129
check_mode_val <- function(mode) {
130-
if (rlang::is_missing(mode) || length(mode) != 1 || !is.character(mode))
130+
if (rlang::is_missing(mode) || length(mode) != 1 || !is.character(mode)) {
131131
rlang::abort("Please supply a character string for a mode (e.g. `'regression'`).")
132+
}
132133
invisible(NULL)
133134
}
134135

@@ -141,8 +142,11 @@ check_spec_mode_val <- function(cls, mode) {
141142
glue::glue_collapse(glue::glue("'{spec_modes}'"), sep = ", ")
142143
)
143144

144-
if (is.null(mode)) rlang::abort(compatible_modes)
145-
else if (!(mode %in% spec_modes)) rlang::abort(compatible_modes)
145+
if (is.null(mode)) {
146+
rlang::abort(compatible_modes)
147+
} else if (!(mode %in% spec_modes)) {
148+
rlang::abort(compatible_modes)
149+
}
146150

147151
invisible(NULL)
148152
}

R/arguments.R

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,9 @@ set_args <- function(object, ...) {
7676
#' @rdname set_args
7777
#' @export
7878
set_mode <- function(object, mode) {
79-
if (rlang::is_missing(mode)) mode <- NULL
79+
if (rlang::is_missing(mode)) {
80+
mode <- NULL
81+
}
8082
mode <- mode[1]
8183
check_spec_mode_val(class(object)[1], mode)
8284
object$mode <- mode

R/engines.R

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,9 @@ check_engine <- function(object) {
1616
object$engine <- avail_eng[1]
1717
rlang::warn(glue::glue("`engine` was NULL and updated to be `{object$engine}`"))
1818
} else {
19-
if (!is.character(object$engine) | length(object$engine) != 1)
19+
if (!is.character(object$engine) | length(object$engine) != 1) {
2020
rlang::abort("`engine` should be a single character value.")
21+
}
2122
}
2223
if (!(object$engine %in% avail_eng)) {
2324
rlang::abort(
@@ -95,7 +96,9 @@ set_engine <- function(object, engine, ...) {
9596
rlang::abort("`object` should have class 'model_spec'.")
9697
}
9798

98-
if (rlang::is_missing(engine)) engine <- NULL
99+
if (rlang::is_missing(engine)) {
100+
engine <- NULL
101+
}
99102
object$engine <- engine
100103
object <- check_engine(object)
101104

0 commit comments

Comments
 (0)