Issues with blavaan #203
strengejackelint-changed-files.yaml
on: pull_request
lint-changed-files
/
lint-changed-files
2m 26s
Annotations
1 error and 7 warnings
|
lint-changed-files / lint-changed-files
Process completed with exit code 31.
|
|
lint-changed-files / lint-changed-files:
R/bayesfactor_models.R#L180
file=R/bayesfactor_models.R,line=180,col=28,[keyword_quote_linter] Only quote targets of extraction with $ if necessary, i.e., if the name is not a valid R symbol (see ?make.names). Use backticks to create non-syntactic names, or use [[ to extract by string.
|
|
lint-changed-files / lint-changed-files:
R/bayesfactor_models.R#L349
file=R/bayesfactor_models.R,line=349,col=28,[keyword_quote_linter] Only quote targets of extraction with $ if necessary, i.e., if the name is not a valid R symbol (see ?make.names). Use backticks to create non-syntactic names, or use [[ to extract by string.
|
|
lint-changed-files / lint-changed-files:
R/bayesfactor_models.R#L449
file=R/bayesfactor_models.R,line=449,col=34,[keyword_quote_linter] Only quote targets of extraction with $ if necessary, i.e., if the name is not a valid R symbol (see ?make.names). Use backticks to create non-syntactic names, or use [[ to extract by string.
|
|
lint-changed-files / lint-changed-files:
R/bayesfactor_models.R#L568
file=R/bayesfactor_models.R,line=568,col=7,[return_linter] Use implicit return behavior; explicit return() is not needed.
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-blavaan.R#L82
file=tests/testthat/test-blavaan.R,line=82,col=18,[implicit_assignment_linter] Avoid implicit assignments in function calls. For example, instead of `if (x <- 1L) { ... }`, write `x <- 1L; if (x) { ... }`.
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-blavaan.R#L89
file=tests/testthat/test-blavaan.R,line=89,col=20,[implicit_assignment_linter] Avoid implicit assignments in function calls. For example, instead of `if (x <- 1L) { ... }`, write `x <- 1L; if (x) { ... }`.
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-blavaan.R#L90
file=tests/testthat/test-blavaan.R,line=90,col=3,[expect_identical_linter] Use expect_identical(x, y) by default; resort to expect_equal() only when needed, e.g. when setting ignore_attr= or tolerance=.
|