data_arrange() gets a by argument
#1132
strengejackelint-changed-files.yaml
on: pull_request
lint-changed-files
/
lint-changed-files
1m 25s
Annotations
1 error and 5 warnings
|
lint-changed-files / lint-changed-files
Process completed with exit code 31.
|
|
lint-changed-files / lint-changed-files:
R/data_arrange.R#L47
file=R/data_arrange.R,line=47,col=7,[if_not_else_linter] Prefer `if (A) x else y` to the less-readable `if (!A) y else x` in a simple if/else statement.
|
|
lint-changed-files / lint-changed-files:
R/data_arrange.R#L54
file=R/data_arrange.R,line=54,col=31,[unnecessary_lambda_linter] Pass .data_arrange directly as a symbol to lapply() instead of wrapping it in an unnecessary anonymous function. For example, prefer lapply(DF, sum) to lapply(DF, function(x) sum(x)).
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-data_arrange.R#L78
file=tests/testthat/test-data_arrange.R,line=78,col=15,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(seq_len(n), m, ...).
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-data_arrange.R#L102
file=tests/testthat/test-data_arrange.R,line=102,col=15,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(seq_len(n), m, ...).
|
|
lint-changed-files / lint-changed-files:
tests/testthat/test-data_arrange.R#L122
file=tests/testthat/test-data_arrange.R,line=122,col=13,[sample_int_linter] sample.int(n, m, ...) is preferable to sample(seq_len(n), m, ...).
|