Merge remote-tracking branch 'refs/remotes/origin/main' #44
Annotations
11 warnings and 1 notice
|
flir
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L38
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=38,col=5,[captured <<- list(...)] <<- can have hard-to-predict behavior; prefer assigning to a specific environment instead (with assign() or <-).
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L24
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=24,col=14,[rpkgkit:::add_caller_to_cli] Operator `:::` is undesirable. It accesses non-exported functions inside packages. Code relying on these is likely to break in future versions of the package because the functions are not part of the public interface and may be changed or removed by the maintainers without notice. Use public functions via :: instead.
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L21
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=21,col=5,[captured <<- list(...)] Avoid undesirable operators `<<-` and `->>`. They assign outside the current environment in a way that can be hard to reason about. Prefer fully-encapsulated functions wherever possible, or, if necessary, assign to a specific environment with assign(). Recall that you can create an environment at the desired scope with new.env().
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L21
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=21,col=5,[captured <<- list(...)] <<- can have hard-to-predict behavior; prefer assigning to a specific environment instead (with assign() or <-).
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L15
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=15,col=3,[expect_equal(captured[[1L]], "[my_func()]: Hello world")] Use expect_identical(x, y) by default; resort to expect_equal() only when needed, e.g. when setting ignore_attr= or tolerance=.
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L7
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=7,col=14,[rpkgkit:::add_caller_to_cli] Operator `:::` is undesirable. It accesses non-exported functions inside packages. Code relying on these is likely to break in future versions of the package because the functions are not part of the public interface and may be changed or removed by the maintainers without notice. Use public functions via :: instead.
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L4
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=4,col=5,[captured <<- list(...)] Avoid undesirable operators `<<-` and `->>`. They assign outside the current environment in a way that can be hard to reason about. Prefer fully-encapsulated functions wherever possible, or, if necessary, assign to a specific environment with assign(). Recall that you can create an environment at the desired scope with new.env().
|
|
flir:
tests/testthat/test-add_caller_to_cli.R#L4
file=/Users/runner/work/rpkgkit/rpkgkit/tests/testthat/test-add_caller_to_cli.R,line=4,col=5,[captured <<- list(...)] <<- can have hard-to-predict behavior; prefer assigning to a specific environment instead (with assign() or <-).
|
|
flir:
R/import-standalone-purrr.R#L103
file=/Users/runner/work/rpkgkit/rpkgkit/R/import-standalone-purrr.R,line=103,col=5,[stopifnot(all(lengths == 1L | lengths == n))] Use stopifnot(x) instead of stopifnot(all(x)). stopifnot(x) runs all() 'under the hood' and provides a better error message in case of failure.
|
|
flir:
R/import-standalone-purrr.R#L101
file=/Users/runner/work/rpkgkit/rpkgkit/R/import-standalone-purrr.R,line=101,col=16,[map_int(.x = args, .f = length)] Use lengths() to find the length of each element in a list.
|
|
flir
The macos-latest label will migrate to macOS 26 beginning June 15, 2026. For more information see https://github.com/actions/runner-images/issues/14167
|