Unverified Commit e08cffba authored by Nikita Titov's avatar Nikita Titov Committed by GitHub
Browse files

[R-package] Specify concrete linting rule that is being ignored via `nolint` mark (#5300)

* Update make-r-def.R

* Update make-r-def.R

* Update build_r.R

* Update utils.R

* Update install.libs.R
parent 2f5baa3d
......@@ -226,7 +226,7 @@ lgb.check.wrapper_param <- function(main_param_name, params, alternative_kwarg_v
#' @importFrom parallel detectCores
lgb.get.default.num.threads <- function() {
if (requireNamespace("RhpcBLASctl", quietly = TRUE)) { # nolint
if (requireNamespace("RhpcBLASctl", quietly = TRUE)) { # nolint: undesirable_function
return(RhpcBLASctl::get_num_cores())
} else {
msg <- "Optional package 'RhpcBLASctl' not found."
......
......@@ -24,7 +24,7 @@ message(sprintf("Creating '%s' from '%s'", OUT_DEF_FILE, IN_DLL_FILE))
.pipe_shell_command_to_stdout <- function(command, args, out_file) {
has_processx <- suppressMessages({
suppressWarnings({
require("processx") # nolint
require("processx") # nolint: undesirable_function
})
})
if (has_processx) {
......@@ -71,7 +71,7 @@ invisible(file.remove(OBJDUMP_FILE))
# see https://www.cs.colorado.edu/~main/cs1300/doc/mingwfaq.html
start_index <- which(
grepl(
pattern = "[Ordinal/Name Pointer] Table" # nolint
pattern = "[Ordinal/Name Pointer] Table" # nolint: non_portable_path
, x = objdump_results
, fixed = TRUE
)
......
......@@ -37,7 +37,7 @@ inst_dir <- file.path(R_PACKAGE_SOURCE, "inst", fsep = "/")
on_windows <- .Platform$OS.type == "windows"
has_processx <- suppressMessages({
suppressWarnings({
require("processx") # nolint
require("processx") # nolint: undesirable_function
})
})
if (has_processx && on_windows) {
......
......@@ -146,7 +146,7 @@ if (length(parsed_args[["make_args"]]) > 0L) {
on_windows <- .Platform$OS.type == "windows"
has_processx <- suppressMessages({
suppressWarnings({
require("processx") # nolint
require("processx") # nolint: undesirable_function
})
})
if (has_processx && on_windows) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment