The ggplot2 argument is linewidth -- should we instead go in the other direction to be consistent with the core plotting framework? #661
strengejackelint-changed-files.yaml
on: pull_request
lint-changed-files
/
lint-changed-files
1m 56s
Annotations
1 error and 2 warnings
|
lint-changed-files / lint-changed-files
Process completed with exit code 31.
|
|
lint-changed-files / lint-changed-files:
R/plot.estimate_density.R#L19
file=R/plot.estimate_density.R,line=19,col=44,[nzchar_linter] Use !nzchar(x) instead of x == "". Note that unlike nzchar(), EQ coerces to character, so you'll have to use as.character() if x is a factor. Whenever missing data is possible, please take care to use nzchar(., keepNA = TRUE); nzchar(NA) is TRUE by default.
|
|
lint-changed-files / lint-changed-files:
R/plot.check_predictions.R#L301
file=R/plot.check_predictions.R,line=301,col=3,[function_argument_linter] Arguments without defaults should come before arguments with defaults.
|