Package review changes #3
lint-changed-files.yml
on: pull_request
lint-changed-files
1m 37s
Annotations
1 error and 11 warnings
lint-changed-files
Process completed with exit code 31.
|
lint-changed-files
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
lint-changed-files:
R/demographics.R#L184
file=R/demographics.R,line=184,col=16,[fixed_regex_linter] This regular expression is static, i.e., its matches can be expressed as a fixed substring expression, which is faster to compute. Here, you can use "-" with fixed = TRUE.
|
lint-changed-files:
R/demographics.R#L207
file=R/demographics.R,line=207,col=16,[fixed_regex_linter] This regular expression is static, i.e., its matches can be expressed as a fixed substring expression, which is faster to compute. Here, you can use "-" with fixed = TRUE.
|
lint-changed-files:
R/demographics.R#L486
file=R/demographics.R,line=486,col=3,[object_usage_linter] local variable 'valid_major_codes' assigned but may not be used
|
lint-changed-files:
R/endemic_channel.R#L147
file=R/endemic_channel.R,line=147,col=1,[trailing_whitespace_linter] Trailing whitespace is superfluous.
|
lint-changed-files:
R/endemic_channel.R#L148
file=R/endemic_channel.R,line=148,col=5,[spaces_left_parentheses_linter] Place a space before left parenthesis, except in a function call.
|
lint-changed-files:
R/endemic_channel.R#L149
file=R/endemic_channel.R,line=149,col=3,[brace_linter] Opening curly braces should never go on their own line and should always be followed by a new line.
|
lint-changed-files:
R/endemic_channel.R#L379
file=R/endemic_channel.R,line=379,col=7,[if_not_else_linter] In a simple if/else statement, prefer `if (A) x else y` to the less-readable `if (!A) y else x`.
|
lint-changed-files:
R/spatiotemporal.R#L12
file=R/spatiotemporal.R,line=12,col=81,[line_length_linter] Lines should not be more than 80 characters. This line is 83 characters.
|
lint-changed-files:
R/spatiotemporal.R#L34
file=R/spatiotemporal.R,line=34,col=74,[T_and_F_symbol_linter] Use TRUE instead of the symbol T.
|
lint-changed-files:
R/spatiotemporal.R#L35
file=R/spatiotemporal.R,line=35,col=46,[undesirable_function_linter] Function "sapply" is undesirable. As an alternative, use vapply() with an appropriate `FUN.VALUE=` argument to obtain type-stable simplification.
|