HenrikBengtsson / matrixStats

R package: Methods that Apply to Rows and Columns of Matrices (and to Vectors)
https://cran.r-project.org/package=matrixStats
202 stars 33 forks source link

CONSISTENCY: Don't use full stop for error messages #257

Closed HenrikBengtsson closed 1 month ago

HenrikBengtsson commented 2 months ago

We sometimes use a full stop (period at the end) of error messages, but not always, e.g.

src/colOrderStats.c:    error("Argument 'which' must be a single number.");
src/colOrderStats.c:    error("Argument 'which' must be a numeric number.");
src/colOrderStats.c:    error("Argument 'rows' must not contain missing value");
src/colOrderStats.c:    error("Argument 'cols' must not contain missing value");

We should not use full stop in error messages.