Merge pull request #955 from strengejacke/strengejacke/issue954 #50
Annotations
11 warnings
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
lint:
R/color_utils.R#L57
file=R/color_utils.R,line=57,col=3,[object_overwrite_linter] 'col' is an exported object from package 'base'. Avoid re-using such symbols.
|
lint:
R/color_utils.R#L60
file=R/color_utils.R,line=60,col=77,[T_and_F_symbol_linter] Use FALSE instead of the symbol F.
|
lint:
R/helpfunctions.R#L107
file=R/helpfunctions.R,line=107,col=5,[unnecessary_nesting_linter] Simplify this condition by using 'else if' instead of 'else { if.
|
lint:
R/helpfunctions.R#L174
file=R/helpfunctions.R,line=174,col=13,[undesirable_function_linter] Avoid undesirable function "structure". As an alternative, Use `class<-`, `names<-`, and `attr<-` to set attributes.
|
lint:
R/helpfunctions.R#L219
file=R/helpfunctions.R,line=219,col=23,[redundant_equals_linter] Using == on a logical vector is redundant. Well-named logical vectors can be used directly in filtering. For data.table's `i` argument, wrap the column name in (), like `DT[(is_treatment)]`.
|
lint:
R/helpfunctions.R#L244
file=R/helpfunctions.R,line=244,col=17,[fixed_regex_linter] Use "0" with fixed = TRUE here. This regular expression is static, i.e., its matches can be expressed as a fixed substring expression, which is faster to compute.
|
lint:
R/helpfunctions.R#L252
file=R/helpfunctions.R,line=252,col=8,[indentation_linter] Hanging indent should be 43 spaces but is 8 spaces.
|
lint:
R/helpfunctions.R#L252
file=R/helpfunctions.R,line=252,col=36,[infix_spaces_linter] Put spaces around all infix operators.
|
lint:
R/helpfunctions.R#L252
file=R/helpfunctions.R,line=252,col=57,[infix_spaces_linter] Put spaces around all infix operators.
|
lint:
R/helpfunctions.R#L252
file=R/helpfunctions.R,line=252,col=78,[infix_spaces_linter] Put spaces around all infix operators.
|